Zsh Mailing List Archive
Messages sorted by: Reverse Date, Date, Thread, Author

Re: [PATCH] Fix ENV handling in sh/ksh emulation



Aside that this patch also included two other hunks changing
isset(INTERACTIVE) to the equivalent "interact" macro instead.

This makes them all the same in init.c, except for one use of
unset(INTERACTIVE) instead of !interact.  However, other files
don't seem to be consistent about making use of this macro,
nor even about whether unset() vs. !isset() is chosen.

I'm not suggesting we go to any special effort to clean this up,
but I'd rather not sneak this kind of thing into unrelated diffs
(at least, not without mentioning it) because, along with things
like whitespace clean-up, it makes it difficult to tell from a
delta what changes are semantically meaningful.



Messages sorted by: Reverse Date, Date, Thread, Author