Build warning: there are no arguments to ‘_’ that depend on a template parameter

[Olorin]

subj

[Steven Sheehy (steven-sheehy)]

./dcpp/Streams.h:140: warning: there are no arguments to ‘_’ that depend on a template parameter, so a declaration of ‘_’ must be available

That warning has been there forever and is not that easy to fix. But marking confirmed anyway to track it and in the hope that someone else fixes it. :)

[Steven Sheehy (steven-sheehy)]

Committed a fix to dcplusplus.

[poy (poy)]

that '_' is a #define normally replaced at compile time; that the compiler is actually trying to match it as some valid token shows that the file that #defines '_', DCPlusPlus.h, has not been included.
this is most likely a case of including stdinc.h but not DCPlusPlus.h; i suggest you modify your precompiled headers to include both stdinc.h & DCPlusPlus.h (see the top of win32/stdafx.h).

[Steven Sheehy (steven-sheehy)]

We do include stdinc.h before DCPlusPlus.h. This error started occurring after we merged .705 or 707 in awhile ago. We had to add -fpermissive in order to ignore the error and just make it a warning. I know '_' is gettext and is already defined by the time Streams.h is processed. I know this since if I add #define _(x) x it warns that I'm redefining '_'. The issue is more nuanced than include order:

http://gcc.gnu.org/onlinedocs/gcc/Name-lookup.html

[Steven Sheehy (steven-sheehy)]

Hmm...nevermind. It is include order. :)

Since we don't (and won't) define BUILDING_DCPP for gui files, we need to include gettext.h before DCPlusPlus.h. Sorry for the mess.

[Olorin (vivliofika)]

It still there... :(

[Steven Sheehy (steven-sheehy)]

It's only fixed in linuxdcpp-i18n branch for now.