[Virtio-fs] [PATCH 1/6] disas: add G_GNUC_PRINTF to gstring_printf

[Stefan Weil]

Am 19.12.22 um 14:02 schrieb Daniel P. Berrangé:
> Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> ---
>   disas.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/disas.c b/disas.c
> index 94d3b45042..31df8f5b89 100644
> --- a/disas.c
> +++ b/disas.c
> @@ -239,6 +239,7 @@ void target_disas(FILE *out, CPUState *cpu, target_ulong code,
>       }
>   }
>   
> +G_GNUC_PRINTF(2, 3)
>   static int gstring_printf(FILE *stream, const char *fmt, ...)
>   {
>       /* We abuse the FILE parameter to pass a GString. */

The current code uses a different pattern:

static RETURN_TYPE G_GNUC_PRINTF(2, 3) function(argument list)

So I would have expected

static int G_GNUC_PRINTF(2, 3)
gstring_printf(FILE *stream, const char *fmt, ...)

Does that matter? Do we care for different patterns?

Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE08C21D5677450AD.asc
Type: application/pgp-keys
Size: 6409 bytes
Desc: OpenPGP public key
URL: <http://listman.redhat.com/archives/virtio-fs/attachments/20221219/82f7787f/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/virtio-fs/attachments/20221219/82f7787f/attachment-0001.sig>