Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Mathias Stearn <[email protected]>
  • Loading branch information
kneth and RedBeard0531 authored Dec 21, 2022
1 parent a12cb37 commit 3d13f43
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/js_subscriptions.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -139,10 +139,10 @@ template <typename T>
void SubscriptionClass<T>::get_name(ContextType ctx, ObjectType this_object, ReturnValue& return_value)
{
auto sub = get_internal<T, SubscriptionClass<T>>(ctx, this_object);
auto name = sub->name;
const auto& name = sub->name;

if (name) {
return_value.set(name);
return_value.set(*name);
}
else {
return_value.set_null();
Expand Down

0 comments on commit 3d13f43

Please sign in to comment.