#include <afxcontrolbars.h> is not compatible with #define NOMINMAX by BongoVR

Hello,

Thanks for the feedback. This issue has been fixed in MFC for the next major release of Visual Studio. The implementation of the methods has been moved from the header files into the respective implementation files, per your suggestion.

Pat Brenner
Visual C++ Libraries Development

Hello,

thanks for sharing your ideas.

Unfortunately, this does not help. std::min is a template assuming that its two arguments are of the same type. On the other hand, the min macro does not care about argument types as long as the operators work; and they do for e. g. int/UINT etc.

I tried using such a workaround before, and this required me to adjust the types in the header code or to explcitly resolve ambiguities, like changing to min<int>(...
This does more harm than it serves.
I can understand that the MFC headers are not supposed to define the min/max macros again and again. So, a better resolution would be to remove the code using these macros from the header altogether and put it into the respective implementation files. As far as I remember, most of these statements occur in virtual functions. To some people, declaring virtual functions inline is a questionable practice anyway.
How about considering this change for the next MFC version?

Hello,

Thanks for the report. This issue is by design in MFC. MFC requires definitions of min and max. Certainly there is no reason to duplicate the macro definitions in the MFC headers, since the same definitions can be obtained by simply not defining NOMINMAX.

If you wish to build your MFC project with NOMINMAX set, you can instead use the following workaround in your stdafx.h file:

#define NOMINMAX
#include <algorithm>
using std::min;
using std::max;

This will provide alternate definitions so that your project will build successfully. I hope this helps.

Pat Brenner
Visual C++ Libraries Development

Thanks for your feedback.
We are routing this issue to the appropriate group within the Visual Studio Product Team for triage and resolution. These specialized experts will follow-up with your issue.

The attached afxcontrolbars.zip contains a patched afxcontrolbars.h header file which corrects this issue.

The attached project in Test2010.zip shows this problem.

Thank you for reporting this issue. Unfortunately, we are unable to reproduce the issue with the steps you provided.

Could you please provide us with a sample project?

It would be greatly appreciated if you could provide us this information as quickly as possible.

Thank you,

Thank you for your feedback, we are currently reviewing the issue you have submitted. If this issue is urgent, please contact support directly(http://support.microsoft.com)