+1 for fixing nxstyle (or configuring another tool like Clang Format
<http://clang.llvm.org/docs/ClangFormat.html>)

That would make it a lot easier to submit PRs that are in the right format,
at least :)

There is no one dedicated to maintaining nxstyle right now.  I wrote the original*, but there was once a plan for Haitao Liu to take that over.  Others (YAMT) and been making good contributions.  So I would not expect any snappy response to nxstyle problems right now.

Greg

* Just a little CYA and history.  nsstyle starting out as a tiny hack tool that I used to check a few things in files.  It got re-used a few times and grew and now it is a key program in the workflow.  It is unfortunate fact that the tool is woefully under designed.  Provided that we set up some good validation tests, I really think it should be redesigned to at least carefully reviewed to make sure that it can actually support the things that we expect from it.

It is a dumb tool, it knows nothing about C syntax.  I don't think it really needs fully understand C syntax, but it does need to understand the context of things better.  Currently it is just a collection of heuristics that spots a few landmarks and makes interfaces from comparison of some patterns.  So it is not very capable.



Reply via email to