patchRe: [PATCH] imap-send: add option to mark sent messages as read or unread

> On 23 Jul 2025, at 11:22 PM, Junio C Hamano <gitster@pobox.com> wrote:
> 
> Junio C Hamano <gitster@pobox.com> writes:
> 
>> 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.