KeyError raised about cppclean HOT 7 CLOSED

Could you format your code in triple backticks (as per Markdown)? The above currently seems a bit scrambled.

Thanks

from cppclean.

Hi,
Sorry, I haven't seen my "+" have been turned in bullets and code was totally unreadable, I edited my post, please tell me if it's enough.
Thanks.

from cppclean.

Can you post the project producing this error or is it private ? Can you try to find a smaller test case ?

Looking at the code, I don't see how we can end up in this case.I'd like to be able to debug this crash to see if your suggestion is enough or if we need a more complex fix.

from cppclean.

Hi,
I managed to reproduce a small test case, and, after several tests to be able to reproduce it with a smallest possible file, it seems that what is leading to this issue is: we have a function under two namespaces (NS1::NS2), defined in an interface and implemented in another class and taking a const reference to an object named as second namespace (NS2).
Couldn't attach zip file then just put those two code in files named as specified in a same folder and run cppclean on it:

• If.h:

#ifndef IF_H_
#define IF_H_

namespace NS1
{
namespace NS2
{
    virtual void func(const NS2& p_ns2) = 0;
}
}

#endif

• Impl.h:

#ifndef IMPL_H_
#define IMPL_H_

#include "If.h"

namespace NS1
{
namespace NS2
{
    virtual void func(const NS2& p_ns2);
}
}

#endif

from cppclean.

Thanks for the testcase. I didn't knew you can use the same name for a class and a namespace !

from cppclean.

I didn't knew too, but afterwards I thought, we can name either namespace or class pretty much as we want (in any case with same rules), and bigger the project is more are the odds to come across with both with same name.

from cppclean.

This should be fixed.

from cppclean.