You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I rather expected these two ways of defining an Ident to work the same, but they don't.
Ident = Letter IdChar* WSC
Ident = v:( Letter IdChar*) WSC { v }
WSC = SP* { "" }
The first one includes trailing whitespace, which is unexpected given that WSC is supposed to translate into empty. The generated code goes to all the trouble of acquiring the correct values for each component and then just does a substring from first to last for the returned value.
Not really that useful as it stands. Not a biggie, but annoying.
The text was updated successfully, but these errors were encountered:
The existing generated code looks like this: state.Subject.Substring(startCursor0.Location, len)
It's a simple solution, to treat a sequence as just a chunk of characters, but really a sequence is the concatenation of its components. The aim would be to generate code more like: String.Concat(r0, r1, r2, ...)
But I agree, it would be a breaking change, so it would be best if it could be optional.
I rather expected these two ways of defining an Ident to work the same, but they don't.
The first one includes trailing whitespace, which is unexpected given that WSC is supposed to translate into empty. The generated code goes to all the trouble of acquiring the correct values for each component and then just does a substring from first to last for the returned value.
Not really that useful as it stands. Not a biggie, but annoying.
The text was updated successfully, but these errors were encountered: