gnu/linux (gentoo) + gcc/g++3.3.6

[basic]

I've had trouble compiling svn trunk with gcc 3.3.6 (gentoo stable). Here's a patch of the changes I made to get compiling. The changes are for:
src/sdk/configmanager.cpp -- gcc 3.3.6 doesn't like wxString().Mid() ...
src/sdk/macrosmanager.h -- missing wx/filename.h include


[attachment deleted by admin]

[basic]

here's another patch. This one is for src/sdk/scriptingmanager.h #include <angelscript.h>. It causes problem when compiling src/src/app.cpp that #include <scriptingmanager.h> and can't find angelscript.h

[attachment deleted by admin]

[me22]

Oops, seems I put my 3 recent threads in the wrong forum.
Mine about the AngelScript issue: http://forums.next.codeblocks.org/index.php?topic=1528.0
Mine about the ConfigManager: http://forums.next.codeblocks.org/index.php?topic=1527.0
Mine about the missing includes: http://forums.next.codeblocks.org/index.php?topic=1530.0

Your angelscript fix may be better, I'm not sure.  I didn't try to compile the angelscript section with my fix.
I think my ConfigManager fix is better, for reasons made clear in the thread.
And I found 2 missing includes in addition to the wxFileName one.

Now we just need to keep poking them to get these applied

[mandrav]

Now we just need to keep poking them to get these applied

How often do you update SVN?

[me22]

Yay!  Thanks =)

* me goes off to finish the new codeblocks-svn ebuild

[me22]

Gah, you fixed it wrong :x

It now says:
configmanager.cpp: In member function `void ConfigManager::Read(const
   wxString&, ConfigManagerContainer::IntToStringMap*)':
configmanager.cpp:1133: error: call of overloaded `wxString(const char*)' is
   ambiguous
/usr/include/wx-2.6/wx/string.h:670: error: candidates are:
   wxString::wxString(const wxWCharBuffer&) <near match>
/usr/include/wx-2.6/wx/string.h:643: error:
   wxString::wxString(wchar_t, unsigned int) <near match>
/usr/include/wx-2.6/wx/string.h:640: error:
   wxString::wxString(const wxString&) <near match>
/usr/include/wx-2.6/wx/string.h:632: error:
   wxString::wxString(int) <near match>

You changed it to:
wxString(curr->Value()).Mid(1).ToLong(&tmp);
which is wrong.  That kind of thing -- getting a wxString from tinyXML -- is precisely what _U() was added to do.  It works fine in non-unicode, because wxChar is a char, but in unicode it tries to make a wchar_t wxString from a char*, which it can't do without more help.

src/sdk/configmanager.cpp line 1133 should be
_U(curr->Value()).Mid(1).ToLong(&tmp);

[mandrav]

Gah, you fixed it wrong :x

And is this a reason to be mad?
Try updating again...

[rickg22]

Hmmm I'm starting to see a pattern here...

me22: GAH! A bug!
Mandrav: Fixed. Update your SVN.
me22: GAH! Your bugfix has a bug!
Mandrav: Fixed. Update your SVN.

Apologies to Ceniza for his 'i'm starting to see a pattern here' posts.

[yop]

Deja vu

[me22]

Gah, you fixed it wrong :x

And is this a reason to be mad?
Try updating again...

It's no reason to be mad, just it makes it seem like you didn't read my rather detailed post about what needed to be changed and why.

Gah! It compiled now!

I've got the codeblocks-svn ebuild done too : http://bugs.gentoo.org/show_bug.cgi?id=89533 =)