Developer forums (C::B DEVELOPMENT STRICTLY!) > Development

Keybinder not compiling after commir r8564

(1/2) > >>

daniloz:
Hi All,

I've found that after updating past r8564, the keybinder plugin is not compiling anymore. The error is

--- Code: ---..\..\..\.objs\plugins\contrib\keybinder\cbkeybinder.o: In function `ZN11cbKeyBinder21MergeAcceleratorTableEb':
C:/Work/codeblocks_trunk/src/plugins/contrib/keybinder/cbkeybinder.cpp:856: undefined reference to `GetFullMenuPath(int)'
..\..\..\.objs\plugins\contrib\keybinder\keybinder.o: In function `ZNK5wxCmd4SaveEP12wxConfigBaseRK8wxStringb':
C:/Work/codeblocks_trunk/src/plugins/contrib/keybinder/keybinder.cpp:696: undefined reference to `GetFullMenuPath(int)'

--- End code ---

I've track the problem to the anonymous namespace introduced in menuutils.cpp, line 137, which causes the function GetFullMenuPath() (and FindMenuIdUsingFullMenuPath()) to be "local" to menuutils.cpp

--- Code: ---namespace
{
// ----------------------------------------------------------------------------
wxString GetFullMenuPath(int id)
// ----------------------------------------------------------------------------
{ ...
...

--- End code ---

Pecan:
The change was made in 8563 (Killerbot).
It indeed will not compile.

What's up Killer?

Miguel Gimenez:
Also, in commit 8561 a bug slipped in projectconfigurationpanel.cpp:

for ( size_t i=0; i<Categories.Count(); i++ )

became

for ( size_t cat = 0; cat < Categories.Count(); cat )

The ++ operator was missed, so the loop never ends.

oBFusCATed:
Great fun  ;D

killerbot:

--- Quote from: mgimenez on November 15, 2012, 05:09:05 pm ---Also, in commit 8561 a bug slipped in projectconfigurationpanel.cpp:

for ( size_t i=0; i<Categories.Count(); i++ )

became

for ( size_t cat = 0; cat < Categories.Count(); cat )

The ++ operator was missed, so the loop never ends.


--- End quote ---
fixed

Navigation

[0] Message Index

[#] Next page