Skip to content

Commit

Permalink
misc: Grammar fixes regarding in the comment explaining why the prior…
Browse files Browse the repository at this point in the history
… fix was made
  • Loading branch information
Col-E committed Jan 10, 2025
1 parent 552630c commit c9aa7b6
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -76,8 +76,10 @@ public JvmMethodPrinter(Method method) {
// where the variable is first assigned (by several instructions). So the code assigning the value
// is outside the variable scope, and usage of the then assigned variable is in-scope.
//
// We'll just do a blunt check if the variable is assigned between the last label and the supposed start.
// If so we'll say that this cope is not valid. This result in some valid label ranges being discarded,
// We'll just do a blunt check to see if the variable is assigned between the last label and
// the supposed starting offset of this local. If we find that the variable is used before the start
// of the local we will just not acknowledge this variable as a valid candidate to use when picking
// variable names. This will result in some valid label ranges being discarded,
// but it is safer to more aggressively discard than to fall into the trap mentioned above.
int priorLabelOffset = findPriorLabelOffset(code, start);
if (isVarUsedInRange(code, index, priorLabelOffset, start))
Expand Down

0 comments on commit c9aa7b6

Please sign in to comment.