[all-commits] [llvm/llvm-project] cf05de: [lldb][NFC] Refactor printing of short options in ...

David Spickett via All-commits all-commits at lists.llvm.org
Tue May 3 08:18:56 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cf05de7168b0ae2eaffdd6ab5a7c982bfa6da5c6
      https://github.com/llvm/llvm-project/commit/cf05de7168b0ae2eaffdd6ab5a7c982bfa6da5c6
  Author: David Spickett <david.spickett at linaro.org>
  Date:   2022-05-03 (Tue, 03 May 2022)

  Changed paths:
    M lldb/source/Interpreter/Options.cpp

  Log Message:
  -----------
  [lldb][NFC] Refactor printing of short options in help

Instead of building a set twice for optional and required,
build a set for each while walking the options once.

Then take advantage of set being sorted meaning we don't
have to enforce the upper/lower order ourselves.

Just cleaned up the formatting on the later loops.
Combined the if conditions and used a single line if.

Depends on D123501

Reviewed By: jingham

Differential Revision: https://reviews.llvm.org/D123502




More information about the All-commits mailing list