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

From: Junio C Hamano <[email-masked]>
Date: Wed, 23 Jul 2025 10:52:35 -0700

Junio C Hamano <[email-masked]> writes:

> Aditya Garg <[email-masked]> 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.