Skip to content

Commit 5d21408

Browse files
etrclaude
andcommitted
TASK-036 review: collapse duplicate route_entry construction (finding 8)
Use aggregate initialization {methods, handler, is_prefix=false} to construct route_entry in insert_fresh_v2_entry, replacing two identical 3-line field-assignment blocks with a single-expression construct per case arm. No behavior change; structure is already correct. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent a11e5ee commit 5d21408

1 file changed

Lines changed: 2 additions & 8 deletions

File tree

src/detail/webserver_routes.cpp

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -213,18 +213,12 @@ void webserver_impl::insert_fresh_v2_entry(const detail::http_endpoint& idx,
213213
// upsert_v2_param_route, never here.
214214
break;
215215
case route_tier_kind::exact: {
216-
detail::route_entry entry;
217-
entry.methods = methods;
218-
entry.handler = std::move(shim);
219-
entry.is_prefix = false;
216+
detail::route_entry entry{methods, std::move(shim), /*is_prefix=*/false};
220217
exact_routes_.emplace(idx.get_url_complete(), std::move(entry));
221218
break;
222219
}
223220
case route_tier_kind::regex_: {
224-
detail::route_entry entry;
225-
entry.methods = methods;
226-
entry.handler = std::move(shim);
227-
entry.is_prefix = false;
221+
detail::route_entry entry{methods, std::move(shim), /*is_prefix=*/false};
228222
regex_routes_.push_back(
229223
{idx.get_url_complete(), std::move(*tier.re), std::move(entry)});
230224
break;

0 commit comments

Comments
 (0)