paraslash/http_send.c, branch nextparaslash - network audio streaming toolshttps://git.tue.mpg.de/cgit/paraslash/atom?h=next2026-03-17T22:45:43ZSwitch to SPDX identifiers.2026-03-17T22:45:43ZAndre Nollmaan@tuebingen.mpg.de2026-03-15T14:57:29Zurn:sha1:ea0cecf96e08c489fdcc94d046019f4ce46f58e2
Generated with
sed -i 's|Copyright.*Andre Noll.*|SPDX-License-Identifier: GPL-2.0 */|g' *.c *.h
followed by manually tweaking the result a bit. No license change intended.
Move user_list.[ch] into server.[ch].2026-03-01T13:44:14ZAndre Nollmaan@tuebingen.mpg.de2026-02-13T20:28:13Zurn:sha1:81a135b2cfee5c3d54c30c8ffea601c7a24fbf6d
This works because only the server links with user_list.o. Since struct user
contains a list head, list.h must now be included *before* server.h, which
requires some trivial header shuffling in the .c files that include server.h.
http_send.c: Simplify format_url().2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-19T19:47:19Zurn:sha1:054935492d28c01578c093fb43982b40dc85cc84
All callers pass an element of the listen_addresses[] array of struct
sender_status, and these elements are never NULL.
Move format_url() from net.c to http_send.c.2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-18T19:45:10Zurn:sha1:2735ca56c3b1177fec6f0ab167cf4b82baf185da
It's only needed there.
Merge acl.c into http_send.c.2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-18T15:40:49Zurn:sha1:b5ef894b6b7d67ae2a0e8159ea0a0da58541007f
Another small file whose functions are only called from http_send.c.
This makes all acl functions static and dedoxifies them.
Merge chunk_queue.c into http_send.c.2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-18T15:00:31Zurn:sha1:b7b74c5fcf14cee58f166243b16ff86107d6c41b
The former is only ~150 lines long and has no users outside of http_send.c. The
merge allows us to mark the chunk queue related functions static, and some
pointless doxygen comments can go away as well.
http_send.c: Move header_sent to struct private_http_sender_data.2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-17T16:37:00Zurn:sha1:30ec45ebf56dce703272fe5c3d2e56f93bdb0cd1
The udp sender does not need it. Convert it to bool and initialize it because
the private_data structure of the http sender is allocated without zeroing.
http_send.c: Move chunk queue into struct private_http_sender_data.2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-17T16:32:12Zurn:sha1:3c023c4ccd10e903de70d0d62951105597a1b9ca
The udp sender does not need it, so it should not be part of the
sender-independent sender_client structure.
Since shutdown_client() needs to derive the private_http_sender_data
structure from the ->private_data pointer of the sender_client pointer, the
declaration of struct private_http_sender_data has to move up. Since this
structure depends on the http_client_status enumeration, the declaration of
the enumeration needs to move up as well.
struct sender: Remove unused sched pointer argument of ->post_monitor().2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-16T18:58:35Zurn:sha1:96078388afd4e9573174f40981d9952a3da17a21
Only the http sender implements ->post_monitor(), and does not use
the argument.
struct sender: Remove unused chunks_sent parameter of ->send().2026-01-26T22:16:18ZAndre Nollmaan@tuebingen.mpg.de2025-09-16T18:54:50Zurn:sha1:f616d44a11995ed1eefffd69d34def8f7a4802c6
Only the http sender sets ->send() (to http_send()), and this function
ignores the parameter.