Re: [Mesa-dev] [PATCH] util: Fix -Wimplicit-function-declaration for ffs with GCC
Dear José Jose Fonseca schrieb am 09.12.2011 19:33: > - Original Message - >> Dear mesa-dev list, >> Kai Wasserbäch schrieb am 09.12.2011 12:59: >>> AFAICS GCC was missed from the last #elif clause. With this patch >>> applied the warning is gone. >>> >>> Signed-off-by: Kai Wasserbäch >>> --- >>> There certainly other warnings thrown during a build, but I saw >>> this one >>> scrolling by, while checking on a build. Hence I went to fix it. >>> >>> src/gallium/auxiliary/util/u_math.h |2 +- >>> 1 files changed, 1 insertions(+), 1 deletions(-) >>> >>> [...] > > The system headers should normally define thi already. > > "man ffs" says it is defined in strings.h > > Does adding > > #include > > to u_math.h fixes the issue? yes, that seems to fix this too. Kind regards, Kai Wasserbäch -- Kai Wasserbäch (Kai Wasserbaech) E-Mail: k...@dev.carbon-project.org signature.asc Description: OpenPGP digital signature ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] [PATCH] util: Fix -Wimplicit-function-declaration for ffs with GCC
- Original Message - > Dear mesa-dev list, > Kai Wasserbäch schrieb am 09.12.2011 12:59: > > AFAICS GCC was missed from the last #elif clause. With this patch > > applied the warning is gone. > > > > Signed-off-by: Kai Wasserbäch > > --- > > There certainly other warnings thrown during a build, but I saw > > this one > > scrolling by, while checking on a build. Hence I went to fix it. > > > > src/gallium/auxiliary/util/u_math.h |2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > [...] The system headers should normally define thi already. "man ffs" says it is defined in strings.h Does adding #include to u_math.h fixes the issue? Jose ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] [PATCH] util: Fix -Wimplicit-function-declaration for ffs with GCC
Dear mesa-dev list, Kai Wasserbäch schrieb am 09.12.2011 12:59: > AFAICS GCC was missed from the last #elif clause. With this patch > applied the warning is gone. > > Signed-off-by: Kai Wasserbäch > --- > There certainly other warnings thrown during a build, but I saw this one > scrolling by, while checking on a build. Hence I went to fix it. > > src/gallium/auxiliary/util/u_math.h |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > [...] I just noticed, I haven't pointed out, that I don't have commit access for Mesa and somebody else would need to commit this, if that patch is accepted. Kind regards, Kai Wasserbäch -- Kai Wasserbäch (Kai Wasserbaech) E-Mail: k...@dev.carbon-project.org signature.asc Description: OpenPGP digital signature ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
[Mesa-dev] [PATCH] util: Fix -Wimplicit-function-declaration for ffs with GCC
AFAICS GCC was missed from the last #elif clause. With this patch applied the warning is gone. Signed-off-by: Kai Wasserbäch --- There certainly other warnings thrown during a build, but I saw this one scrolling by, while checking on a build. Hence I went to fix it. src/gallium/auxiliary/util/u_math.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/util/u_math.h b/src/gallium/auxiliary/util/u_math.h index f908341..068a8d1 100644 --- a/src/gallium/auxiliary/util/u_math.h +++ b/src/gallium/auxiliary/util/u_math.h @@ -364,7 +364,7 @@ unsigned ffs( unsigned u ) return i; } -#elif defined(__MINGW32__) || defined(PIPE_OS_ANDROID) +#elif defined(__MINGW32__) || defined(PIPE_OS_ANDROID) || defined(PIPE_CC_GCC) #define ffs __builtin_ffs #endif -- 1.7.7.3 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev