balazske added a comment.

Adding special test functions is not as easy, then `StreamState` should be 
accessible from another checker. It could be added to the same file, or new 
file but then moving the data structures into header is needed. At least for 
short-term it is more simple to add a stream function that generates error 
state (`fseek` is applicable or other). This is why the `fseek` patch is better 
to be included in this change (but makes it more complicated). Otherwise this 
revision must be added without tests for state observer functions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75682



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

Reply via email to