diff options
| author | Lukas Rössler <pontomedon@gmail.com> | 2012-05-21 08:29:21 -0600 |
|---|---|---|
| committer | Brian Paul <brianp@vmware.com> | 2012-05-21 08:29:23 -0600 |
| commit | 98398a3e02eca8706a21f58736246a8bb3bd3f92 (patch) | |
| tree | c5f6323ec98f6232785553d5e93efca7e02d0c54 | |
| parent | 89f3a504bfd1747ae6f7e1ef70cde0d26f2037db (diff) | |
| download | glu-98398a3e02eca8706a21f58736246a8bb3bd3f92.tar.gz | |
glu: fix two Clang warnings
This patch removes two Clang warnings in GLU:
The first one seems to be an actual bug in mapdesc.cc: Clang complains
that sizeof(dest) will return the size of REAL*[MAXCOORDS], instead of
the intended REAL[MAXCOORDS][MAXCOORDS]. The second one is just
cosmetic because Clang doesn't like extra parentheses.
NOTE: This is a candidate for the 8.0 branch
Reviewed-by: Brian Paul <brianp@vmware.com>
| -rw-r--r-- | src/libnurbs/internals/mapdesc.cc | 2 | ||||
| -rw-r--r-- | src/libnurbs/internals/nurbstess.cc | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/libnurbs/internals/mapdesc.cc b/src/libnurbs/internals/mapdesc.cc index d59f8fd..0a96c5f 100644 --- a/src/libnurbs/internals/mapdesc.cc +++ b/src/libnurbs/internals/mapdesc.cc @@ -90,7 +90,7 @@ Mapdesc::setBboxsize( INREAL *mat ) void Mapdesc::identify( REAL dest[MAXCOORDS][MAXCOORDS] ) { - memset( dest, 0, sizeof( dest ) ); + memset( dest, 0, sizeof( REAL ) * MAXCOORDS * MAXCOORDS ); for( int i=0; i != hcoords; i++ ) dest[i][i] = 1.0; } diff --git a/src/libnurbs/internals/nurbstess.cc b/src/libnurbs/internals/nurbstess.cc index 68dfd95..e477a8c 100644 --- a/src/libnurbs/internals/nurbstess.cc +++ b/src/libnurbs/internals/nurbstess.cc @@ -505,7 +505,7 @@ NurbsTessellator::do_pwlcurve( O_pwlcurve *o_pwlcurve ) o_pwlcurve->owner = currentCurve; } - if( (inCurve == 2) ) + if( inCurve == 2 ) endcurve(); } |