Hi,

* Bert Wesarg wrote (2008-01-04 18:12):
>> 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.

I'm probably missing something, but couldn't you just put the
transiency check on top of the tree?

Alternatively, to check the state model, you could leave it near the
bottom and add a debug print to get warnings about locked transient
files.


Thorsten
-- 
A future startup with no patents of its own will be forced to pay whatever
price the giants choose to impose. That price might be high: Established
companies have an interest in excluding future competitors.
    - Bill Gates

Attachment: pgpLarR8j4HxE.pgp
Description: PGP signature

-- 
NEdit Develop mailing list - [email protected]
http://www.nedit.org/mailman/listinfo/develop

Reply via email to