Bugs?

[maxblagaj]

I think it should be logical "&&" here.


--- src/plugins/debuggergdb/gdb_driver.cpp   (revision 3643)
+++ src/plugins/debuggergdb/gdb_driver.cpp   (working copy)
@@ -325,7 +325,7 @@
         // are only checking for the CDprefix not any furthur correctness
         if(path.GetChar(0)== g_EscapeChars)
         {
-            while(i<path.Len()& (path.GetChar(i)==g_EscapeChars))
+            while(i<path.Len() && (path.GetChar(i)==g_EscapeChars))
             {
                 // get character
                 escCount+=1;



SquirrelError is a structure, not a class. And it look like wxWidgets macros like EVT_MENU() etc have problems with multiple inheritance sometimes when wxEvtHandler is not the first base class.

--- src/include/scriptingmanager.h   (revision 3643)
+++ src/include/scriptingmanager.h   (working copy)
@@ -14,7 +14,7 @@
 #include "menuitemsmanager.h"
 #include <wx/intl.h>
 
-class SquirrelError;
+struct SquirrelError;
 
 /** @brief Provides scripting in Code::Blocks.
   *
@@ -42,7 +42,7 @@
   * The templated type denotes the function's return type. Also note that the
   * function name is not unicode (we 're not using Squirrel in unicode mode).
   */
-class DLLIMPORT ScriptingManager : public Mgr<ScriptingManager>, public wxEvtHandler
+class DLLIMPORT ScriptingManager : public wxEvtHandler, public Mgr<ScriptingManager>
 {
         friend class Mgr<ScriptingManager>;
         wxCriticalSection cs;

[maxblagaj]

Another one?? Variable tgtIdx is used without been initialized.

debuggergdb.cpp:868

        if (!project->BuildTargetValid(tgt, false))
        {
            int tgtIdx = project->SelectTarget(tgtIdx);
            if (tgtIdx == -1)
            {
                msgMan->Log(m_PageIndex, _("canceled"));
                return 3;
            }
            target = project->GetBuildTarget(tgtIdx);
        }
        else
            target = project->GetBuildTarget(tgt);