jimingham wrote: I agree with Dave that verbs would be more naturally follow the intent of the return. Since the return is telling the caller: either "You can reuse the children you have cached" or "you must refetch the children". So `eReuse`, `eRefetch` would produce code like:
if (something.Update() == ChildCacheState::eRefetch) { // Blow away the cached children and refetch. } That seems pretty natural. https://github.com/llvm/llvm-project/pull/80167 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits