Jean-Marc Lasgouttes <[EMAIL PROTECTED]> writes:
| When compiling text.C I get the following warning:
>
| ../../lyx-devel/src/text.C: In method `int LyXText::rightMargin (const
| Paragraph &, const Buffer &, const Row &) const':
| ../../lyx-devel/src/text.C:615: warning: value computed is not used
>
| Indeed rightMargine reads:
>
| int LyXText::rightMargin(Paragraph const & par,
| Buffer const & buf, Row const &) const
| {
| LyXTextClass const & tclass = buf.params().getLyXTextClass();
| LyXLayout_ptr const & layout = par.layout();
>
| return PAPER_MARGIN
| + font_metrics::signedWidth(tclass.rightmargin(),
| tclass.defaultfont());
| + font_metrics::signedWidth(layout->rightmargin,
| tclass.defaultfont())
| * 4 / (par.getDepth() + 4);
| }
>
>
| How long do you need to spot the error?
two seconds
| Who designed a language
| allowing such stupid constructs?
Kernigan, Richice and Stroustrup
| I am not sure what the fix is, so I'll let someone who knows fix the
| function.
beats me... drop the ';' delete the next line?
Hmm... but what has depth to do wiht right margin? delete the last
three lines... unless you can see obvious bugs already present.
What does CVS Blame say? :-)
--
Lgb
