April White:

> I have corrected the bug and uploaded the correction to the same file as
> above.

   The CmdQueue code looks like an abandoned experiment as it doesn't
appear to be linked in.

   I don't much like specifying all the string modes explicitly like
("savebefore", "savebefore:1", "savebefore:yes", "savebefore:2", and
"savebefore:no"). Shouldn't this be broken up into recognition of the
mode name and the value as it was previously? This is a good example
of not following "don't repeat yourself" with the chance of failing to
spell all instances the same or only updating some when making a
change.

   Methods shouldn't end with ';' after the '}'.

   Seems to work OK for running commands but I haven't tried many variations.

   A warning from Borland (line number may differ from your version)
Warning W8004 .\SciTEWin.cxx 817: 'cancelled' is assigned a value that
is never used in function SciTEWin::ExecuteOne(const Job &,bool &)

   Neil

_______________________________________________
Scite-interest mailing list
[email protected]
http://mailman.lyra.org/mailman/listinfo/scite-interest

Reply via email to