79

I have the following template declaration:

template <typename T>
   void IterTable(int&                       rIdx,
                  std::vector<double>&       rVarVector,
                  const std::vector<T>&      aTable,
                  const T                    aValue,
                  T              aLowerBound = -(std::numeric_limits<T>::max()), //illegal token on right side of '::' shows here
                  bool                       aLeftOpen = true) const;

Which throws the illegal token error as noted, on the line with "-(std::numeric_limits::max())". I got this code from some old linux source that I'm trying to compile on Windows. Any idea what the issue is?

Edit: It also fails using min(), and the compiler output is:

Error   92  error C2589: '::' : illegal token on right side of '::' c:\projects\r&d\prepaydll\include\cfcdefault.h  216 PrepayDLL

Error   93  error C2059: syntax error : '::'    c:\projects\r&d\prepaydll\include\cfcdefault.h  216 PrepayDLL

Line 216, is the line previously mentioned.

5
  • Works for me. Have you #include <limits>?
    – kennytm
    Apr 1, 2010 at 15:08
  • 1
    It would be oh so helpful if the compiler told us what this token was. In some regards, modern C++ compilers still suck. (And this kind of error message isn’t limited to just one compiler.) Apr 1, 2010 at 15:12
  • Try saving/viewing the preprocessor output. (/E or /P).
    – stusmith
    Apr 1, 2010 at 15:26
  • Have you checked wether <limits> is included? Apr 1, 2010 at 18:53
  • 1
    No need to include any limits library or no need to define NOMINMAX. Refer the solution here : stackoverflow.com/a/22023122/2535756 Just add parenthesis to (std::max) or (std::min)
    – Ganesh S
    Oct 13, 2019 at 7:09

3 Answers 3

177

My guess is that max has been made a macro. This happens at some point inside windows.h.

Define NOMINMAX prior to including to stop windows.h from doing that.

EDIT:

I'm still confident this is your problem. (Not including <limits> would result in a different error). Place #undef max and #undef min just before the function and try again. If that fixes it, I was correct, and your NOMINMAX isn't being defined properly. (Add it as a project setting.)

You can also prevent macro expansion by: (std::numeric_limits<T>::max)().


On a side note, why not do std::numeric_limits<T>::min() instead of negating the max?

10
  • 14
    Or, if you can't use NOMINMAX for some reason, you can prevent the macro expansion by parenthesizing the function name: (std::numeric_limits<T>::max)(). This is very, very ugly, but can be necessary in some cases. Apr 1, 2010 at 19:06
  • 3
    For twos-complement integers (which are pretty much all you get anymore), the minimum is one less than the negation of the maximum. So, I agree with your side note: why negate the max? Apr 1, 2010 at 19:26
  • 3
    The (std::numeric_limits<T>::max)() worked for me just fine. A very handy post! Mar 28, 2018 at 13:21
  • 4
    It's astounding how much trouble the windows.h definition of max/min causes. Jun 28, 2019 at 16:53
  • 1
    @danelliottster Indeed. I cannot use a standard library function because of a stupid lowercase definition.
    – Burak
    Nov 20, 2020 at 16:29
5

Looks like you need to:

#include <limits>

1
  • This is the right answer and it solved build issues.
    – Ganesh S
    Oct 13, 2019 at 7:07
-1

I wrote a "test harness" with a trivial struct containing your method declaration (and nothing else), and #included <limits> and <vector>, and invoked (and thus instantiated) the method with T being int, and it compiled just fine, both on Visual Studio 2008 Express on Windows Vista and with GCC 4.2.4 on Linux 2.6.

I suggest trying to build only a minimal amount of code with the "problem" in it, and if that actually does build, add back in the rest of your project until it breaks, then you'll know what caused it.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.