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)