Fix compiler warnings (gcc 6.1).

Some of these warnings were benign (code formatting), but one of them
showed a potential bug (zero divide in test/fraciewer.cxx).

Fixed warnings:

 2 suggest parentheses around operand of '!' or change '&' to '&&' or '!' to '~' [-Wparentheses]
 1 this 'while' clause does not guard... [-Wmisleading-indentation]
 1 this 'for' clause does not guard... [-Wmisleading-indentation]
 1 this 'else' clause does not guard... [-Wmisleading-indentation]


git-svn-id: file:///fltk/svn/fltk/branches/branch-1.3@11849 ea41ed52-d2ee-0310-a9c1-e6b18d33e121
This commit is contained in:
Albrecht Schlosser 2016-07-29 09:23:44 +00:00
parent 9721373219
commit a51a940890
4 changed files with 20 additions and 14 deletions

View File

@ -3,7 +3,7 @@
//
// Base Browser widget class for the Fast Light Tool Kit (FLTK).
//
// Copyright 1998-2010 by Bill Spitzak and others.
// Copyright 1998-2016 by Bill Spitzak and others.
//
// This library is free software. Distribution and use rights are outlined in
// the file "COPYING" which should have been included with this file. If this
@ -714,9 +714,10 @@ int Fl_Browser_::handle(int event) {
if (type()==FL_HOLD_BROWSER) {
switch (Fl::event_key()) {
case FL_Down:
while ((l = item_next(l)))
if (item_height(l)>0) {select_only(l, when()); break;}
return 1;
while ((l = item_next(l))) {
if (item_height(l)>0) {select_only(l, when()); break;}
}
return 1;
case FL_Up:
while ((l = item_prev(l))) {
if (item_height(l)>0) {

View File

@ -38,10 +38,14 @@ static void drawhand(double ang,const float v[][2],Fl_Color fill,Fl_Color line)
{
fl_push_matrix();
fl_rotate(ang);
fl_color(fill); fl_begin_polygon();
int i; for (i=0; i<4; i++) fl_vertex(v[i][0],v[i][1]); fl_end_polygon();
fl_color(line); fl_begin_loop();
for (i=0; i<4; i++) fl_vertex(v[i][0],v[i][1]); fl_end_loop();
fl_color(fill);
fl_begin_polygon();
int i; for (i=0; i<4; i++) fl_vertex(v[i][0],v[i][1]);
fl_end_polygon();
fl_color(line);
fl_begin_loop();
for (i=0; i<4; i++) fl_vertex(v[i][0],v[i][1]);
fl_end_loop();
fl_pop_matrix();
}

View File

@ -491,14 +491,14 @@ void Fl_Table::recalc_dimensions() {
int scrollsize = Fl::scrollbar_size();
#endif
// Second pass: Check for interference
if ( !hideh & hidev ) { hidev = (( table_h - tih + scrollsize ) <= 0 ); }
if ( !hidev & hideh ) { hideh = (( table_w - tiw + scrollsize ) <= 0 ); }
if ( !hideh && hidev ) { hidev = (( table_h - tih + scrollsize ) <= 0 ); }
if ( !hidev && hideh ) { hideh = (( table_w - tiw + scrollsize ) <= 0 ); }
// Determine scrollbar visibility, trim ti[xywh]/to[xywh]
if ( hidev ) { vscrollbar->hide(); }
if ( hidev ) { vscrollbar->hide(); }
else { vscrollbar->show(); tiw -= scrollsize; tow -= scrollsize; }
if ( hideh ) { hscrollbar->hide(); }
if ( hideh ) { hscrollbar->hide(); }
else { hscrollbar->show(); tih -= scrollsize; toh -= scrollsize; }
}
}
// Resize the child table
table->resize(tox, toy, tow, toh);
table->init_sizes();

View File

@ -483,8 +483,9 @@ static void normalize(GLfloat v[3])
if (d == 0)
fprintf(stderr, "Zero length vector in normalize\n");
else
else {
v[0] /= d; v[1] /= d; v[2] /= d;
}
}
/* calculates a normalized crossproduct to v1, v2 */