qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [6324] Return -errno on write failure (Gleb Natapov)
Date: Fri, 16 Jan 2009 12:49:04 -0600	[thread overview]
Message-ID: <4970D6A0.5090609@codemonkey.ws> (raw)
In-Reply-To: <2431978291-BeMail@laptop>

François Revol wrote:
>> So BeOS is not ANSI C !
>>     
>
> And so, well, what ?
> It won't change any time soon.
> Besides, there is no reason a language stantard should dictate such a 
> runtime thing...
> And for what I've seen from it, it's not consistent with itself, saying 
> non-zero on a line, positive on the next one.
>
> Btw, was it available freely at the time ? I mean don't expect people 
> to comply with something you have to pay for. That's what you get. 
> (hint, POSIX drafts)
>
> It's not like any other OS I've seen doesn't violate some standard.
>
> Still, everything has been fine for a decade and suddenly people start 
> doing this kind of tricks out of the blue. It's not like considering 
> errno can be !=0 is orthogonal to the standard, so supporting BeOS 
> itself would't make the code itself violate ANSI C.
>   

If a full set of patches was posted that to QEMU that made the latest 
SVN compile on BeOS with a free compiler, I would think we would 
evaluate them and determine how intrusive the changes were compared to 
the value of supporting BeOS and apply them if appropriate.  Avoiding 
-errno, IMHO, is a fair trade off to support another platform even if 
there is a niche user base.

That said, such patches have not been posted.  I don't think it's wise 
to get into a habit of avoiding things because some random platform that 
QEMU doesn't support today doesn't support said feature.

So I'm not inclined to change code that conforms to ANSI unless someone 
puts the work into supporting BeOS in mainline QEMU and that said work 
doesn't require very invasive changes.

Regards,

Anthony Liguori

> François.
>
>
>   

  reply	other threads:[~2009-01-16 18:49 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-15 20:44 [Qemu-devel] [6324] Return -errno on write failure (Gleb Natapov) Anthony Liguori
2009-01-15 21:20 ` François Revol
2009-01-15 22:15   ` M. Warner Losh
2009-01-15 22:33     ` François Revol
2009-01-16  7:31   ` Gleb Natapov
2009-01-16 17:37   ` Ian Jackson
2009-01-16 18:36     ` François Revol
2009-01-16 18:49       ` Anthony Liguori [this message]
2009-01-16 19:17         ` François Revol
2009-01-16 19:31           ` Anthony Liguori
2009-01-16 19:41             ` François Revol
2009-01-19 11:37       ` Ian Jackson
2009-01-19 18:52         ` François Revol
2009-01-19 19:47           ` Lennart Sorensen
2009-01-19 20:31             ` François Revol
2009-01-19 20:03           ` M. Warner Losh

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4970D6A0.5090609@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).