Skip to content

Commit 7823b27

Browse files
committed
Use bare pointer instead of unique_ptr, simpler and faster (slightly)
1 parent ab4a25a commit 7823b27

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

src/native/common/include/runtime-base/timing-internal.hh

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ namespace xamarin::android {
6565
time_point start;
6666
time_point end;
6767
TimingEventKind kind;
68-
std::unique_ptr<std::string> more_info;
68+
std::string *more_info = nullptr;
6969
};
7070

7171
class FastTiming;
@@ -202,7 +202,7 @@ namespace xamarin::android {
202202
message.append ("] "sv);
203203

204204
append_event_kind_description (event.kind, message);
205-
if (event.more_info && !event.more_info->empty ()) {
205+
if (event.more_info != nullptr && !event.more_info->empty ()) {
206206
message.append (event.more_info->c_str (), event.more_info->length ());
207207
}
208208

@@ -234,7 +234,7 @@ namespace xamarin::android {
234234
return;
235235
}
236236

237-
if (skip_log_if_more_info_missing && (!event.more_info || event.more_info->empty ())) {
237+
if (skip_log_if_more_info_missing && (event.more_info == nullptr || event.more_info->empty ())) {
238238
return;
239239
}
240240

@@ -305,7 +305,7 @@ namespace xamarin::android {
305305
return;
306306
}
307307

308-
events[*index].more_info = std::make_unique<std::string> (str.get (), str.length ());
308+
events[*index].more_info = new std::string (str.get (), str.length ());
309309
log (events[*index], false /* skip_log_if_more_info_missing */);
310310
}
311311

@@ -318,7 +318,7 @@ namespace xamarin::android {
318318
return;
319319
}
320320

321-
events[*index].more_info = std::make_unique<std::string> (str);
321+
events[*index].more_info = new std::string (str);
322322
log (events[*index], false /* skip_log_if_more_info_missing */);
323323
}
324324

@@ -331,7 +331,7 @@ namespace xamarin::android {
331331
return;
332332
}
333333

334-
events[*index].more_info = std::make_unique<std::string> (str);
334+
events[*index].more_info = new std::string (str);
335335
log (events[*index], false /* skip_log_if_more_info_missing */);
336336
}
337337

src/native/common/runtime-base/timing-internal.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
#include <chrono>
22

3-
//#include <xamarin-app.hh>
43
#include <runtime-base/android-system.hh>
54
#include <runtime-base/startup-aware-lock.hh>
65
#include <runtime-base/strings.hh>

0 commit comments

Comments
 (0)