[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-sparse
Subject:    Re: [PATCH 04/10] compile-i386.c: don't mix calls to write(2) with stdio
From:       Ramsay Jones <ramsay () ramsay1 ! demon ! co ! uk>
Date:       2014-10-02 10:27:43
Message-ID: 542D289F.9090701 () ramsay1 ! demon ! co ! uk
[Download RAW message or body]

On 30/09/14 03:34, Christopher Li wrote:
> On Sat, Sep 27, 2014 at 9:00 PM, Ramsay Jones
> <ramsay@ramsay1.demon.co.uk> wrote:
>>>> -       write(STDOUT_FILENO, s, strlen(s));
>>>> +       printf("%s", s);
>>>
>>
>> I was only really fixing the compiler warning (in an side-ways fashion).
>> (although I think it is an improvement).
> 
> I would just check the return value and "die" if there is an error to silence
> the error.

Hmmm, I can change the patch to do that if you prefer.

Having said that, given that this is not currently maintained, what
do you think about a patch to remove it?

[Are you hoping someone will pick it up again soon?]

Let me know which patch you would like to see.

ATB,
Ramsay Jones



--
To unsubscribe from this list: send the line "unsubscribe linux-sparse" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]