This little snippet has gotten fairly messy... the "strlen(phpStringDelimiter)" is computed 1, 2, or 3 times in a row at this point. Also, only when it is added to an int does the "static_cast<int>" get used - but the two integer compares are "bare".

Maybe a "const int n = strlen(phpStringDelimiter)" with whatever temp name and cast usage is appropriate would be preferable?

Just a thought as I was testing a new install of TortoiseCVS and looking for things to diff... ;)

Robert Roessler
[EMAIL PROTECTED]
http://www.rftp.com
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest

Reply via email to