Steven Myint
Steven Myint
I can't reproduce this: ``` $ cat Foo.h class Blah; class Foo: public Bar{ public: Foo(); virtual bool doSomething(Blah* blah); }; $ ./cppclean Foo.h ```
Your suggestion seems reasonable to me.
Thanks for reporting this. I don't have time to look at this in the near term, but, I've added you as a collaborator to this repository.
I think it would be `pip install cppclean`. I'm not sure what `py` is, but you would need to make sure that the location `pip` installed the `cppclean` executable to...
Oh, I see, this is on Windows. We'll need to upgrade [`setup.py`](https://github.com/myint/cppclean/blob/master/setup.py) to use `setuptools` like for example [`autoflake` does](https://github.com/myint/autoflake/blob/b4824635ecdec24a87c32ad6a1c61f6dce09b785/setup.py).
Interesting! It looks like @r-e-d added [this](https://github.com/myint/cppclean/commit/e7a4645fa25d7beddbfb0ff3978a6285234c6851#diff-c1d30fd80542acff620e02d4e9cec9a7R168) in e7a4645fa25d7beddbfb0ff3978a6285234c6851. But I don't see a rationale for that conditional. Unless there are objections, I think we should remove the conditional.
Or perhaps make it an option that can be configured?
@marmistrz What is contained in `.`?
@r-e-d, probably has a better idea than me on whether this is a good approach.
Thanks! I can merge this in once the functionality in the existing `eradicate` script is somehow merged into this.