From f8bfab5284eb457808fc9294ae7b1b944186aab6 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Tue, 9 Feb 2016 23:38:04 +0100 Subject: [PATCH] Replace out of date comment before wx/windowid.h inclusion No real changes, just remove the nonsensical comment from wx/defs.h which completely lost its meaning since wxWindowID typedef was replaced by a class 8 years ago in cf2810aa394d47a43489d6138edf0497c389a22c. --- include/wx/defs.h | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/include/wx/defs.h b/include/wx/defs.h index d63daf8e86..9a28c6fd3e 100644 --- a/include/wx/defs.h +++ b/include/wx/defs.h @@ -2320,14 +2320,17 @@ enum wxStandardID }; /* ---------------------------------------------------------------------------- */ -/* wxWindowID type (after wxID_XYZ enum, platform detection, and dlimpexp.h) */ +/* wxWindowID type */ /* ---------------------------------------------------------------------------- */ -/* special care should be taken with this type under Windows where the real */ -/* window id is unsigned, so we must always do the cast before comparing them */ -/* (or else they would be always different!). Using wxGetWindowId() which does */ -/* the cast itself is recommended. Note that this type can't be unsigned */ -/* because wxID_ANY == -1 is a valid (and largely used) value for window id. */ +/* + * wxWindowID used to be just a typedef defined here, now it's a class, but we + * still continue to define it here for compatibility, so that the code using + * it continues to compile even if it includes just wx/defs.h. + * + * Notice that wx/windowid.h can only be included after wxID_XYZ definitions + * (as it uses them). + */ #if defined(__cplusplus) && wxUSE_GUI #include "wx/windowid.h" #endif