Latest Tip; WinMain Error

[jrlanglois]

Hi Jules,

Just got the latest tip, and now my project throws a compile error saying ‘WinMain’ : function cannot be overloaded…

In case you’re wondering, I’m of course including Windows.h before the JuceHeader.h since I’ve not run into this issue until today.

[jrlanglois]

For what it’s worth, the last time I pulled the latest tip was last Friday, and I didn’t have issues then.

[TheVinn]

This sounds to me like a mismatch between the Unicode declaration of WinMain using wide strings versus the ANSI declaration.

[jrlanglois]

The issue was created/started at this commit:

Revision: 18cfadf898aa1678dc6b092d4dd314ed082d567f
Author: jules
Date: 25/06/2012 6:05:39 AM
Message:
Refactored the handling of command-line args: added new static methods JUCEApplication::getCommandLineParameters() and getCommandLineParameterArray().

---

Modified : modules/juce_core/native/juce_linux_Files.cpp
Modified : modules/juce_core/native/juce_mac_Files.mm
Modified : modules/juce_core/native/juce_win32_Threads.cpp
Modified : modules/juce_core/threads/juce_Process.h
Modified : modules/juce_events/messages/juce_ApplicationBase.h
Modified : modules/juce_gui_basics/application/juce_Application.cpp
Modified : modules/juce_gui_basics/application/juce_Application.h
Modified : modules/juce_gui_basics/application/juce_Initialisation.h
Modified : modules/juce_gui_basics/native/juce_android_Windowing.cpp
Modified : modules/juce_gui_basics/native/juce_ios_Windowing.mm
Modified : modules/juce_gui_basics/native/juce_win32_Windowing.cpp

[jrlanglois]

https://github.com/julianstorer/JUCE/commit/18cfadf898aa1678dc6b092d4dd314ed082d567f

I think the issue lies within changes (aka Removals) to this file specifically: modules/juce_gui_basics/application/juce_Initialisation.h

[TheVinn]

New

int __stdcall WinMain (void*, void*, const char*, int)

Old

int WINAPI WinMain (HINSTANCE, HINSTANCE, LPSTR, int)

[jules]

Ah, I couldn’t remember why I had two different versions of that code hanging around, so cleaned it up. I guess the reason the old code was there is because otherwise it breaks if you’ve already included windows.h. Thanks, I’ll sort that out…

[TheVinn]

@jules

Ah, I couldn’t remember why I had two different versions of that code hanging around, so cleaned it up. I guess the reason the old code was there is because otherwise it breaks if you’ve already included windows.h. Thanks, I’ll sort that out…

As it is written, it would seem the declaration of WinMain might break, since it is hard-coded to use a non-unicode string. To declare WinMain correctly for all values of the UNICODE project setting I think you want to use LPTSTR instead of LPSTR.

[jrlanglois]

Thanks Jules.

I must say that I’m confused about the differences between what you have there, and LPSTR and HINSTANCE… I totally thought those were char* and void* respectively…? Or is there some hidden unicode bs flag that affects this stuff?

[jrlanglois]

…And you’ve just answered my question as I submitted my post, TheVinn.

[dnop]

I’m using currently the commit tagged with 2.0.27.
This version does not work with unicode and Windows.h included. When I change the WinMain definition from

int __stdcall WinMain (HINSTANCE, HINSTANCE, const LPTSTR, int)

to

int __stdcall WinMain (HINSTANCE, HINSTANCE, const LPSTR, int)

it works again.

[jules]

Ah, right… yes, it does look like it should just be an LPSTR, thanks!

[Peter_Emanuel_Roos]

Hi all,

I am running also into this issue, with the latest version of Juce (pulled today). Made a simple GUI app with the Introducer.

Is there something that I should explicitly set in the Introducer or in my VC++ 2010 project?

TIA.

Peter

[jrlanglois]

Are you including Windows.h? If so, are you doing so before including JuceHeader.h?

[Peter_Emanuel_Roos]

Yep, thanks - I went through the other topics on this and got it fixed. Great framework, but steep learning curve :-)