I get the following warning:
../../1.5.x/src/TextMetrics.cpp: In member function `lyx::RowMetrics 
lyx::TextMetrics::computeRowMetrics(lyx::pit_type, const lyx::Row&) const':
../../1.5.x/src/TextMetrics.cpp:343: warning: enumeration value `Inline' not 
handled in switch
../../1.5.x/src/TextMetrics.cpp:343: warning: enumeration value `AlignRight' 
not handled in switch

Indeed the code reads:

                    switch(par.getInset(row.pos())->display()) {
                        case Inset::AlignLeft:
                                align = LYX_ALIGN_BLOCK;
                                break;
                        case Inset::AlignCenter:
                                align = LYX_ALIGN_CENTER;
                                break;
                        // other types unchanged (use align)
                        }

Bo (I think it is your code?), you should give all the enum values in
such a switch. Actually, why should AlignRight not be handled?

Also, the indentation/style of this part of the code is not correct.

JMarc

Reply via email to