[Lldb-commits] [PATCH] D112537: [formatters] Add a libstdcpp formatter for set and unify tests across stdlibs

[Danil Stefaniuc via Phabricator via lldb-commits]

danilashtefan added inline comments.


================
Comment at: lldb/test/API/functionalities/data-formatter/data-formatter-stl/generic/set/TestDataFormatterGenericSet.py:45
                 "[3] = 3",
                 "[6] = 6"])
 
----------------
wallace wrote:
> can you add additional assertions like the one you did in https://reviews.llvm.org/D112180 that uses ValueCheck?
I have made check method more generic and less verbose. Before it was specifically checking the "ii" variable, that was a set<int> of size 7. With value check approach we can now test set<string> and others. 

I guess, that I can delete all of the
```
self.expect() 
```
and simply call check method with the expected size. Please, let me know if I should do it or leave it as it is. Thanks


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112537