for_statement init node

[xixiaofinland]

in for_statement, it support these two init node scenarios:

field("init", $.local_variable_declaration),
seq(commaJoined(field("init", $.expression)), ";")

So there can be multiple init: Expression nodes, but this is not parsable in jorje: PA site sample code

public class Me {
  {
    for ('hoho', 'haha'; i < 3;) {
      System.debug(i);
    }
  }
}

[xixiaofinland]

similar case goes to update node, sample code

public class Me {
  {
    for (i = 0; i < 3; i++, i--) {
      System.debug(i);
    }
  }
}

[aheber]

Thanks for adding these. It'll take me a bit to clean them up. I know why it is happening but I'll have to do some work to figure out what is supported by the platform for those slots. Right now they are matching a catch-all and I'll need to narrow that down to only what is valid. (And see where else this would come up)

[xixiaofinland]

no problem, take your time :)

[xixiaofinland]

as this is more loose than jorje, maybe it's OK as it won't cause any trouble.

[aheber]

I do plan to correct this, it is just a lower priority still because it isn't correct but that lack of correctness shouldn't hurt much for now.