On Wednesday 23 October 2002 1:37 pm, Rob Lahaye wrote:
> Angus Leeming wrote:
> >>2001-09-08  Rob Lahaye  <[EMAIL PROTECTED]>
> >>
> >>* FeedbackController.C: separate width as an "int const";
> >
> > Seems a little excessive, as it's used only once...
>
> Fine with me.
> Let me make this remark: in the LyX code I have seen so very
> many cases of "type const var = ...", used only once, that I
> thought it may be a coding policy for better readability.
> Well, indeed it increases the readability.
>
> If I remember well from one of my C++ books, the compiler may
> remove the "type const var = " definitions and plug the value
> into the place instead of the variable. This certainly will
> happen for const variables that are used only once.
>
> So when I see a long expression, that is difficult to
> comprehend, I consider ripping it into one or several const
> defines, to improve readability. Is that alright, or not?

Yes. It's a good thing. You'll find it in the patch when it's applied...
A

Reply via email to