Discussion:
[patch] Combine OutlineIn and OutlineOut blocks
Scott Kostyshak
2014-06-07 06:16:29 UTC
Permalink
Two blocks of code differed very little so I combined them. I'm not
sure what LyX's style norms are about this. I like this because (1)
now it is not easy to change in one place and forget about the other
and (2) it's easy to see exactly where the code differs between the
two.

Opinions?

Scott
Richard Heck
2014-06-07 16:34:55 UTC
Permalink
Post by Scott Kostyshak
Two blocks of code differed very little so I combined them. I'm not
sure what LyX's style norms are about this. I like this because (1)
now it is not easy to change in one place and forget about the other
and (2) it's easy to see exactly where the code differs between the
two.
Opinions?
Yes, this always eases maintenance.

Richard
Scott Kostyshak
2014-06-08 07:44:49 UTC
Permalink
Post by Richard Heck
Post by Scott Kostyshak
Two blocks of code differed very little so I combined them. I'm not
sure what LyX's style norms are about this. I like this because (1)
now it is not easy to change in one place and forget about the other
and (2) it's easy to see exactly where the code differs between the
two.
Opinions?
Yes, this always eases maintenance.
In at 4cfc110.

Scott

Loading...