[Lldb-commits] [PATCH] D121155: [lldb] Add --exists flag to `settings set`

2022-03-08 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG8a0fa4db39d8: [lldb] Add --exists flag to `settings set` (authored by JDevlieghere). Herald added a project: LLDB. Changed prior to commit: https://reviews.llvm.org/D121155?vs=413615&id=414016#toc Repo

[Lldb-commits] [PATCH] D121155: [lldb] Add --exists flag to `settings set`

2022-03-08 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. I suggested what seems to me a slightly clearer error message, but the added option seems useful. I wondered briefly about whether it might be better as a setting `settings set ignore-missi

[Lldb-commits] [PATCH] D121155: [lldb] Add --exists flag to `settings set`

2022-03-07 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: jingham, jasonmolenda, mib. Herald added a subscriber: dang. Herald added a project: All. JDevlieghere requested review of this revision. Add a --exists/-e flag to `settings set` that sets the setting if it exists, but doesn't print