@gnikit thanks.

I just noticed a comment on the OP that some names are in more than one list.

That won't break anything, but it will have performance effects.  The lexer has 
to search all lists for every identifier it finds (so all your variable names, 
function names, etc as well as keywords, it doesn't know they are keywords 
until it finds them in a list) so duplicating names makes lists bigger which 
will slow the lexer down searching them, especially for those identifiers not 
in a list (it does a linear search of names with the same start character in 
each list).

Maybe somebody might want to "optimise" it, it won't hurt to delay for a bit 
since there is no release on the horizon.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/3656#issuecomment-1951597787
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/pull/3656/[email protected]>

Reply via email to