[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: wget2 | Extract common HTTP functions (!534)
From: |
@rockdaboot |
Subject: |
Re: wget2 | Extract common HTTP functions (!534) |
Date: |
Fri, 20 Oct 2023 18:15:21 +0000 |
Tim Rühsen started a new discussion on libwget/http.h:
https://gitlab.com/gnuwget/wget2/-/merge_requests/534#note_1612877128
> #define HTTP_STATUS_NOT_FOUND 404
> #define HTTP_STATUS_RANGE_NOT_SATISFIABLE 416
>
> +int wget_http_connection_is_aborted(wget_http_connection *conn);
Overall, the change LGTM.
Are you sure about the `wget_` prefix?
I think the functions are "internal" only, so a prefix like `http_`, `http2_`
should in theory be enough.
WDYT?
--
Reply to this email directly or view it on GitLab:
https://gitlab.com/gnuwget/wget2/-/merge_requests/534#note_1612877128
You're receiving this email because of your account on gitlab.com.