Re: [PATCH] imap-send: add option to mark sent messages as read or unread
Junio C Hamano <gitster@pobox.com> writes:
Show 17 quoted lines
> Aditya Garg <gargaditya08@live.com> writes:
>
>> + if (mark_seen) {
>> + curl_easy_setopt(curl, CURLOPT_UPLOAD_FLAGS, CURLULFLAG_SEEN);
>> + } else {
>> + curl_easy_setopt(curl, CURLOPT_UPLOAD_FLAGS, 0L);
>> + }
>> +
>
> Why so many braces around single-statement blocks?
>
> According to
>
> https://github.com/curl/curl/blob/master/docs/libcurl/symbols-in-versions
>
> CURLULFLAG_SEEN first appeared in 8.13.0; INSTALL says we require 7.61.0
> or later, so this may be OK.It might be obvious, but I meant "may not be OK" here.
Junio C Hamano <gitster@pobox.com>