On Fri, Aug 23, 2024 at 12:14:22AM +0200, Stefano Brivio wrote:
> On Thu, 15 Aug 2024 17:50:22 +0200
> Laurent Vivier <lvivier@redhat.com> wrote:
[snip]

> > +	if (sz_payload) {
> > +		do {
> > +			ret = recv(conn_fd, &vmsg->payload, sz_payload, 0);
> > +		} while (ret < 0 && (errno == EINTR || errno == EAGAIN));
> 
> No need for curly brackets, it's a one-line statement.

Unlike if, while or for, I'm pretty sure the braces are mandatory for
do {} while.

-- 
David Gibson (he or they)	| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you, not the other way
				| around.
http://www.ozlabs.org/~dgibson