[PATCH] D75682: [Analyzer][StreamChecker] Introduction of stream error handling.

[Balogh, Ádám via Phabricator via cfe-commits]

baloghadamsoftware added a comment.

In D75682#1916809 <https://reviews.llvm.org/D75682#1916809>, @balazske wrote:

> What is better? Have this (or similar) change that adds a feature that is used in a later change and is only testable in that later change, or have a bigger change that contains real use of the added features? (This means: Add `fseek` to this change or not.) The error handling is not normally testable if there is no function that generates error state, `fseek` could be one.


I think neither: add the feature (the last enum value and its handling in the functions) in the `fseek()` patch were it is used.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75682