[PATCH] D26481: Make the Error class constructor protected

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 9 16:00:51 PST 2016


mehdi_amini created this revision.
mehdi_amini added a reviewer: lhames.
mehdi_amini added a subscriber: llvm-commits.
Herald added subscribers: danalbert, tberghammer, emaste.

This is forcing to use Error::success(), which is in a wide majority
of cases a lot more readable.


https://reviews.llvm.org/D26481

Files:
  lldb/include/lldb/Target/Process.h
  lldb/source/Commands/CommandObjectDisassemble.cpp
  lldb/source/Commands/CommandObjectExpression.cpp
  lldb/source/Core/ModuleList.cpp
  lldb/source/Host/common/NativeBreakpoint.cpp
  lldb/source/Host/common/NativeBreakpointList.cpp
  lldb/source/Host/common/NativeProcessProtocol.cpp
  lldb/source/Host/common/NativeWatchpointList.cpp
  lldb/source/Host/common/SoftwareBreakpoint.cpp
  lldb/source/Host/posix/FileSystem.cpp
  lldb/source/Host/posix/MainLoopPosix.cpp
  lldb/source/Host/posix/PipePosix.cpp
  lldb/source/Host/windows/LockFileWindows.cpp
  lldb/source/Host/windows/PipeWindows.cpp
  lldb/source/Interpreter/OptionGroupVariable.cpp
  lldb/source/Interpreter/OptionValueString.cpp
  lldb/source/Plugins/DynamicLoader/Hexagon-DYLD/DynamicLoaderHexagonDYLD.cpp
  lldb/source/Plugins/DynamicLoader/POSIX-DYLD/DynamicLoaderPOSIXDYLD.cpp
  lldb/source/Plugins/DynamicLoader/Windows-DYLD/DynamicLoaderWindowsDYLD.cpp
  lldb/source/Plugins/Platform/Android/AdbClient.cpp
  lldb/source/Plugins/Platform/FreeBSD/PlatformFreeBSD.cpp
  lldb/source/Plugins/Platform/Kalimba/PlatformKalimba.cpp
  lldb/source/Plugins/Platform/Linux/PlatformLinux.cpp
  lldb/source/Plugins/Platform/MacOSX/PlatformAppleSimulator.cpp
  lldb/source/Plugins/Platform/MacOSX/PlatformDarwin.cpp
  lldb/source/Plugins/Platform/MacOSX/PlatformMacOSX.cpp
  lldb/source/Plugins/Platform/NetBSD/PlatformNetBSD.cpp
  lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp
  lldb/source/Plugins/Platform/Windows/PlatformWindows.cpp
  lldb/source/Plugins/Platform/gdb-server/PlatformRemoteGDBServer.cpp
  lldb/source/Plugins/Process/FreeBSD/ProcessFreeBSD.cpp
  lldb/source/Plugins/Process/Linux/NativeProcessLinux.cpp
  lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_arm.cpp
  lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_arm64.cpp
  lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_mips64.cpp
  lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_s390x.cpp
  lldb/source/Plugins/Process/Linux/NativeRegisterContextLinux_x86_64.cpp
  lldb/source/Plugins/Process/Linux/NativeThreadLinux.cpp
  lldb/source/Plugins/Process/MacOSX-Kernel/ProcessKDP.cpp
  lldb/source/Plugins/Process/Windows/Live/ProcessWindowsLive.cpp
  lldb/source/Plugins/Process/Windows/MiniDump/ProcessWinMiniDump.cpp
  lldb/source/Plugins/Process/elf-core/ProcessElfCore.cpp
  lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
  lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp
  lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
  lldb/source/Plugins/Process/mach-core/ProcessMachCore.cpp
  lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp
  lldb/source/Target/Platform.cpp
  lldb/source/Target/Process.cpp
  lldb/source/Target/Thread.cpp
  lldb/source/Utility/ModuleCache.cpp
  lldb/tools/lldb-server/lldb-platform.cpp
  lldb/unittests/Utility/ModuleCacheTest.cpp
  llvm/include/llvm/ExecutionEngine/Orc/OrcRemoteTargetClient.h
  llvm/include/llvm/Support/Error.h
  llvm/lib/LTO/LTO.cpp
  llvm/lib/LTO/LTOBackend.cpp
  llvm/lib/Object/Archive.cpp
  llvm/lib/Object/Error.cpp
  llvm/lib/Object/MachOObjectFile.cpp
  llvm/lib/Object/MachOUniversal.cpp
  llvm/lib/ProfileData/Coverage/CoverageMappingReader.cpp
  llvm/tools/dsymutil/BinaryHolder.cpp
  llvm/tools/llvm-ar/llvm-ar.cpp
  llvm/tools/llvm-cxxdump/llvm-cxxdump.cpp
  llvm/tools/llvm-dwp/llvm-dwp.cpp
  llvm/tools/llvm-nm/llvm-nm.cpp
  llvm/tools/llvm-objdump/MachODump.cpp
  llvm/tools/llvm-objdump/llvm-objdump.cpp
  llvm/tools/llvm-readobj/llvm-readobj.cpp
  llvm/tools/llvm-size/llvm-size.cpp
  llvm/tools/llvm-xray/xray-extract.cc
  llvm/tools/sancov/sancov.cc
  llvm/unittests/Support/ErrorTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26481.77416.patch
Type: text/x-patch
Size: 73814 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20161110/68d98f37/attachment.bin>


More information about the llvm-commits mailing list