On Mon, Oct 21, 2002 at 04:03:14PM +0100, John Levon wrote:
> > Concerning coding style (i.e. whitespace and braces) the current code is in
> > pretty good shape. It's certainly much better than a few years ago. And we
> > were talking about this kind of stylistic issues.
> 
>         // set the correct parindent
>         if (row->pos() == 0) {
>                 if ((layout->labeltype == LABEL_NO_LABEL
>                      || layout->labeltype == LABEL_TOP_ENVIRONMENT
>                      || layout->labeltype == LABEL_CENTERED_TOP_ENVIRONMENT
>                      || (layout->labeltype == LABEL_STATIC
>                          && layout->latextype == LATEX_ENVIRONMENT
>                          && ! row->par()->isFirstInSequence()))

Given the mess of code, this is a very sensible way to format it.

>                     && align == LYX_ALIGN_BLOCK
>                     && !row->par()->params().noindent()
>                         // in tabulars and ert paragraphs are never indented!
>                         && (!row->par()->inInset() || 
>!row->par()->inInset()->owner() ||
>                                 (row->par()->inInset()->owner()->lyxCode() != 
>Inset::TABULAR_CODE &&
>                                  row->par()->inInset()->owner()->lyxCode() != 
>Inset::ERT_CODE))

Btw: Do you see where this "checking for special values" leads to?

>                 } else if (layout->labeltype == LABEL_BIBLIO) {
>                         // ale970405 Right width for bibitems
>                         x += bibitemMaxWidth(bview, tclass.defaultfont());
>                 }
>         }

And bibitem should be deleted. [I know, it took Cato a while...]

Andre'

-- 
Those who desire to give up Freedom in order to gain Security,
will not have, nor do they deserve, either one. (T. Jefferson)

Reply via email to