cppclean crash on specific header about cppclean HOT 7 CLOSED

I don't know if it is the cause of the bug, but this seems to use the _lookup_in_all_namespaces() code path. Either way, the end result is that a dict gets stored in the symbol lookup table rather than the expected tuple.

Thanks

from cppclean.

OK, if it helps, I also get the same (I think) crash on a much shorter (186 lines instead of 2282 lines) file.

from cppclean.

This one also crashes in the same way, and is only 144 lines. If I comment out all the explicit instantiations in the file, which are of the form:

template<>
inline
Real WendlandRBF<3,0>::operator()(Real rad) const
{
  if (rad > _rcut) return 0.;

  rad /= _rcut;

  return Utility::pow<2>(1.-rad);
}

then cppclean runs fine, for whatever that's worth.

from cppclean.

Hi,

Any bug report is fine. It's easier to debug if you reduce the testcase like in your last post but a complete file is also ok.

Neither of these files seems to crash after the latest patch. Can you confirm that this is fixed for you ? In this case, we can close this issue.

from cppclean.

Can you confirm that this is fixed for you ? In this case, we can close this issue.

It looks like it's working -- I no longer seem to get the invalid warnings about unused forward declarations and the headers that cppclean was crashing on previously now seem to work. Thanks!

BTW, I just cloned the repo and used the cppclean script there directly. Can I easily pip install the Github version in place of 0.5? I have not used pip much before, unfortunately.

from cppclean.

Something like:

$ pip uninstall cppclean
$ pip install git+https://github.com/myint/cppclean

from cppclean.

Thanks, that worked great.

BTW since the problem seems to be resolved I'm closing the issue...

from cppclean.