[all-commits] [llvm/llvm-project] 1582dd: [lldb] Change more uses of AppendMessageWithFormat...

[David Spickett via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 1582dd9c31d5f0f3d234d165b0a770b8118f6a48
      https://github.com/llvm/llvm-project/commit/1582dd9c31d5f0f3d234d165b0a770b8118f6a48
  Author: David Spickett <david.spickett at arm.com>
  Date:   2026-03-04 (Wed, 04 Mar 2026)

  Changed paths:
    M lldb/source/Commands/CommandObjectApropos.cpp
    M lldb/source/Commands/CommandObjectBreakpointCommand.cpp
    M lldb/source/Commands/CommandObjectPlatform.cpp
    M lldb/source/Commands/CommandObjectSource.cpp
    M lldb/source/Commands/CommandObjectType.cpp
    M lldb/source/Commands/CommandObjectVersion.cpp
    M lldb/source/Commands/CommandObjectWatchpointCommand.cpp
    M lldb/source/Plugins/LanguageRuntime/CPlusPlus/ItaniumABI/ItaniumABILanguageRuntime.cpp

  Log Message:
  -----------
  [lldb] Change more uses of AppendMessageWithFormat to AppendMessageWithFormatv (#184337)

When the message includes a final newline, Formatv can add that for you.

The only unusual change is one place in platform where we need to print
octal. LLVM doesn't have a built in way to do this (see
llvm/include/llvm/Support/FormatProviders.h) and this is probably the
only place in the codebase that wants to. So I decided not to add it
there.

Instead I've put the number info a format adapter with the normal printf
specifier, then put that into the Formatv format.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications