JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.
Nice cleanup
================
Comment at: lldb/source/Host/common/HostNativeThreadBase.cpp:55-56
HostNativeThreadBase::ThreadCreateTrampoline(lldb::thread_arg_t arg) {
- ThreadLauncher::HostThreadCreateInfo *info =
- (ThreadLauncher::HostThreadCreateInfo *)arg;
- llvm::set_thread_name(info->thread_name);
-
- thread_func_t thread_fptr = info->thread_fptr;
- thread_arg_t thread_arg = info->thread_arg;
+ std::unique_ptr<ThreadLauncher::HostThreadCreateInfo> info_up(
+ (ThreadLauncher::HostThreadCreateInfo *)arg);
+ llvm::set_thread_name(info_up->thread_name);
----------------
make_unique maybe?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120321/new/
https://reviews.llvm.org/D120321
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
