[ nedit-Patches-1058246 ] Patch Collection

[Bert Wesarg]

Hi,

On Dec 28, 2007 8:33 PM, Thorsten Haude <yoo at vranx.de> wrote:
> enhanced-move-file_open_hook.patch:
> Why would you want that?
Because I think callbacks should be the last thing to do before the
action is finished.

>
> enhanced-transient.patch:
> All good, but not yet patched in: Are you planning to continue work on
> the incomplete title formatting support? If yes, would it be easier to
> keep the patch seperated from the Collection for now?
The last thing to be done is the status (%S) flag in the window title.
currently the logic is as follows:

if (IS_ANY_LOCKED_IGNORING_USER(lockReasons) && fileChanged)
   status = "read only, modified";
else if (IS_ANY_LOCKED_IGNORING_USER(lockReasons))
   status = "read only";
else if (IS_USER_LOCKED(lockReasons) && fileChanged)
   status = "locked, modified";
else if (IS_USER_LOCKED(lockReasons))
   status = "locked";
else if (fileChanged)
   status = "modified";

I just added as a last but one case this (ie before the last one):

else if (transient)
   status = "transient";

But all the previously checks with the fileChanged conditions are
'wrong' if the file is transient.

I may change these '&& fileChanged' checks to '&& (fileChanged &&
!transient)' which prevents these ", modified" strings in case of
transient files.

Is this a way to go?

Bert