Fixes errors like
src/libnurbs/internals/varray.cc:76:5: error: ISO C++17 does not allow 'register' storage class specifier [-Wregister]
register long ds = sgn(arc->tail()[0] - arc->prev->tail()[0]);
^~~~~~~~~
Signed-off-by: Khem Raj <raj.khem@gmail.com>
Long ago when GLU lived in the Mesa repository, there was "name
mangling" support that allowed you to build Mesa and GLU with all
exported symbols' names "mangled". That is, the symbol names would
typically be prefixed with an 'm' so that they would not collide with
the system's OpenGL implementation.
When GLU was split out into its own repository in 2012 we didn't bring
over the name mangling support. No one has noticed this lack of
functionality since, to my knowledge.
Remove the name mangling header (which was unexpectedly licensed under
the GPL).
This should be on par with the autoconf build. Like autoconf it
defauilts to generating both a shared and static library. I have
intentionally not used the pkg-config template, and used Meson's
pkg-config generator instad. This provides dependency and include
tracking that would otherwise have to be done manually.
Right now the the enableval isn't checked, so the debug options are added
for --enable-debug and --disable-debug, which is quite surprising for a
user. Fix this to properly handle the disable case.
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
pqNewPriorityQ() function creates and setups PriorityQ structure, all
except for the field "order". It is filled later in function
pqInit(). Depending on vertices of polygon which must be tesselated
there possible following situation, pqDeletePriorityQ() is called
right after pqNewPriorityQ() function. pqNewPriorityQ() tries to free
memory using pq->order as pointer, which is unitialized at this point.
Signed-off-by: Brian Paul <brianp@vmware.com>
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>
typedef void (GLAPIENTRYP _GLUfuncptr)(); causes the following warning:
function declaration isn't a prototype.
Signed-off-by: José Fonseca <jfonseca@vmware.com>
Including windows.h was ineffective on MSVC because we define the NOGDI macro,
which skips the wingdi.h include.
Unsetting NOGDI is also a bad idea because it causes all sort of symbol
clashes with SGI code.
The real problem is that WINGDAPI was not being defined, also due to NOGDI,
so simply define it to blank if not done already. This seems to make
everybody happy.
When compiled with the more aggressive compiler warnings such as
-Wshadow and -Wempty-body the libtess code gives a lot more
warnings. This fixes the following issues:
* The 'Swap' macro tries to combine multiple statements into one and
then consume the trailing semicolon by using if(1){/*...*/}else.
This gives warnings because the else part ends up with an empty
statement. It also seems a bit dangerous because if the semicolon
were missed then it would still be valid syntax but it would just
ignore the following statement. This patch replaces it with the more
common idiom do { /*...*/ } while(0).
* 'free' was being used as a local variable name but this shadows the
global function. This has been renamed to 'free_handle'
* TRUE and FALSE were being unconditionally defined. Although this
isn't currently a problem it seems better to guard them with #ifndef
because it's quite common for them to be defined in other headers.
https://bugs.freedesktop.org/show_bug.cgi?id=28845
Signed-off-by: Brian Paul <brianp@vmware.com>
This commit breaks flightgear fgfs. A null pointer is
dereferenced in the function MakeVertex.
This reverts commit 9be414f4b93c763e1aee8f634a13721ae5d95fcd.
