Neil Hodgson wrote:
...
These modifications have changed very fundamental parts of
Scintilla and are likely to have caused new bugs and to have changed
performance so it would be good to see them tested and any bugs
reported.
So far, these changes seem to a) work and b) not break anything - at
least in terms of my uses of SciTE and the Scintilla component in my
own software.
Performance is tougher to gauge, given that an awful lot of things
look fast since I upgraded my dev system from the ol' P-III 1.4 GHz to
the new C2D 2.4 GHz in September... :)
While we are [vaguely] on the subject of correctness, there are 5
warnings that are shown every time I build Scintilla under MSVC (well,
actually only in my own GTK-on-Windows project):
ScintillaGTK(1995):
"empty controlled statement found; is this the intent?"
This could be easily fixed by removing the IsUnicodeMode() test as a
separate "if" statement and placing it at the front of the following
[conditionally compiled] "if" as "!IsUnicodeMode() && ". This would
have the added benefit of not executing code we really don't need to.
PlatGTK.cxx(2286):
PlatGTK.cxx(2359):
PlatGTK.cxx(2367):
PlatGTK.cxx(2391):
"forcing value to bool 'true' or 'false' (performance warning)"
These could all just have "!= FALSE" appended and everyone would be happy.
I would be more than happy to make the described changes.
Robert Roessler
[EMAIL PROTECTED]
http://www.rftp.com
_______________________________________________
Scintilla-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scintilla-interest
