[PATCH] D61015: [LibTooing] Change Transformer's TextGenerator to a partial function.
Ilya Biryukov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 25 06:23:54 PDT 2019
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.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits