MyDeveloperDay added a comment.

> Yes, we are, and as you said, it needs a set of patches to learn the code 
> base enough to contribute, to build up trust. This is easiest when the code 
> base is currently under active development, but many people consider 
> clang-format "good enough" minus bug fixes / new languages (like the C# 
> patch, which I haven't looked at).
> 
> Are you by chance going to be at the Euro LLVM dev meeting? If so, I'd be 
> happy to sit together with you to work on / talk through patches.

Alas I am not. I would however value your input on the C# patch, as I work 
running it through some larger C# code bases I've found some constructs that 
need additional changes. In the meantime my proposal still stands, I'd like to 
help, if that just means doing the leg work of ensuring patches have 
documentation, are formatted etc... but I don't want to railroad this revision 
with that discussion.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D28462/new/

https://reviews.llvm.org/D28462



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to