| Re: [AD] Another Grabber patch |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
Owen Rudge wrote:
> I've attached another patch for the Grabber. This adds a "dirty" flag to the
> Grabber and sets it whenever anything is modified. It doesn't pick up on
> every situation and may say something was modified when it wasn't - someone
> may want to fix that. :-)
I know (from experience) how hard it is to make a 'dirty' flag for a
document in a more complicated program. I also know how annoying it is
for a user if such a thing doesn't work correctly. If you add this,
please make sure it works correctly, or leave it out. Once it's in
people start to depend on it, and if then it doesn't work properly,
(ie leaves cases where it doesn't report correctly) things may go
horribly wrong. If there is no such 'file changed' warning, people
will automatically keep track themselves and be careful about it.
In the spirit of the grabber I wouldn't mind keeping it simple without
such a warning, but if you do add it, please make sure it covers all
cases.
Hein Zelle
>-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-<
Hein Zelle hein@xxxxxxxxxx
http://www.icce.rug.nl/~hein
>-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-<