Anton Khirnov <[email protected]> writes:

> +
> +     flags |= O_NONBLOCK;
> +     ret = fcntl (pipes[i], F_SETFL, flags);
> +     if (ret == -1) {
> +         _notmuch_database_log (db, "Error making the pipe non-blocking: 
> %s\n",
> +                                strerror (errno));
> +         goto CLOSE;
> +     }
> +    }
> +
> +    _filter_attachment_communicate (&pipes[0], &pipes[1],
> +                                 &data, &data_len, db);

I hesitate to mention it, but I think this could reasonably be one line.

I was also annoyed by

>      body = (char *) g_byte_array_free (byte_array, false);
>  

But I realized I probably wrote it ;).

d

_______________________________________________
notmuch mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to