Bugs item #1549613, was opened at 2006-08-30 14:57
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=102439&aid=1549613&group_id=2439
Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Scintilla
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Nobody/Anonymous (nobody)
Assigned to: Nobody/Anonymous (nobody)
Summary: Improper case-insensitive handling in PropSet.cxx (+Fix)
Initial Comment:
Assume keywords contains "TODO" and case-sensitivity is
turned on, then TODO will not be matched with the
current code.
In WordList::InList (and a couple of other functions)
the words returned are ignorant of case-sensitivity.
If a user has case-insensitive turned on and a keyword
that is mixed case (or upper-case in my example) then
this is handline it improperly. I've redefined the
functions InList and InListAbbreviated to be (lines 95-96):
bool InList(const char *s, bool caseInsensitive = false);
bool InListAbbreviated(const char *s, const char
marker, bool caseInsensitive = false);
so that they are aware of the case sensitivity change
(most of the code is already in the functions, so it is
a minimal change). I've attached the new PropSet.cxx
as the changes are substantial enough (I feel). I have
not updated any of the case-insensitive lexers, and
this should probably be done at some point.
batman900 AT gmail DOT com
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=102439&aid=1549613&group_id=2439
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest
