scylladb / seastar

High performance server-side application framework
http://seastar.io
Apache License 2.0
8.38k stars 1.55k forks source link

http: clarify the ownership in the router's doxygen comment #2423

Closed kreuzerkrieg closed 2 months ago

kreuzerkrieg commented 2 months ago

Update the routes's put, add, add_default_handler, add_cookie and del_cookie doxygen documentation

tchaikov commented 2 months ago

the title of the commit message reads:

Update the routes's put, add, add_default_handler, add_cookie and del_cookie doxygen documentation.

update ... documentation, is not accurate enough, i'd suggest put:

http: clarify the ownership in the router's doxygen comment

for two reasons:

tchaikov commented 2 months ago

@scylladb/seastar-maint hello maintainers, could you help review this change?