ilya-biryukov added a comment.

I'd argue it's the server's job to validate the inputs in that case.

The code that landed so far clearly looks like a C++ DSL to describe 
transformations of the source. While it **can** be used a dependency in the 
server-side, I don't see why doing user-input checking should be done by the 
library, rather than the server itself.
User input would have to be transformed into the calls of the C++ API somehow, 
that looks like a proper layer to do the validation.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61015



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

Reply via email to