[Lldb-commits] [lldb] [lldb][NFC] Fix a build failure with MSVC (PR #111231)

Igor Kudrin via lldb-commits lldb-commits at lists.llvm.org
Mon Oct 7 12:20:20 PDT 2024


igorkudrin wrote:

> Looks good, I guess. Is that constructor even necessary, given that it just forwards the argument to the base class, and we already have the forwarding `using` declaration?

The constructor looks redundant at the moment, perhaps it is just a legacy from the early stages of patch #106442. Maybe some other constructors can also be reduced, for example, `Win32Error::Win32Error()`. However, I may not fully understand the intentions of the author of the original patch, so I'd prefer @adrian-prantl to do the cleanup if necessary. My patch is only intended to fix the immediate problem.

https://github.com/llvm/llvm-project/pull/111231


More information about the lldb-commits mailing list