rleigh-codelibre commented on pull request #2:
URL: https://github.com/apache/xerces-c/pull/2#issuecomment-646975989


   @viniul The addition of a test looks fairly low-risk in terms of wider 
impact on the project.
   
   Has the necessary Apache ICLA and/or company CLA been done?
   
   On the integration side, it would be nice to have this added to the 
CMakeLists.txt as for the other tests, and likewise to Makefile.am.  If the 
fuzzing is long-running, I would suggest not adding to CTest so that it isn't 
part of the regular integration testing.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: c-dev-unsubscr...@xerces.apache.org
For additional commands, e-mail: c-dev-h...@xerces.apache.org

Reply via email to