qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Aurelien Jarno <aurelien@aurel32.net>
To: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [4164] MIPS Magnum R4000 machine
Date: Tue, 08 Apr 2008 10:01:39 +0200	[thread overview]
Message-ID: <47FB2663.7030603@aurel32.net> (raw)
In-Reply-To: <47FB12CB.8000400@reactos.org>

Hervé Poussineau a écrit :
> Aurelien Jarno a écrit :
>> Revision: 4164
>>           http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=4164
>> Author:   aurel32
>> Date:     2008-04-07 19:47:14 +0000 (Mon, 07 Apr 2008)
>>
>> Log Message:
>> -----------
>> MIPS Magnum R4000 machine
>  >
>> Added Paths:
>> -----------
>>     trunk/hw/mips_jazz.c
>>     trunk/hw/rc4030.c
>>
> 
> Please be consistent.
> The added files are encoded in unicode, whereas existing ones are not.

Wrong they are not encoded in unicode (should be UTF-16 from your
description on IRC), but in UTF-8. It's your software which convert them
to UTF-16.

Also a few other files are *already* encoded in UTF-8 in the SVN.

> Also, diff.exe on Windows (provided with MSYS) considers unicode files 
> as binary, which means that I can't provide patches for them...

Then you software is buggy.

The only workaround I can see from the software point of view is to
remove the accents from you name in those files, so that they are
changed to plain ASCII. But in that case, please don't submit patches
with non-ASCII chars. Also I won't change other files already in UTF-8.

-- 
  .''`.  Aurelien Jarno	            | GPG: 1024D/F1BCDB73
 : :' :  Debian developer           | Electrical Engineer
 `. `'   aurel32@debian.org         | aurelien@aurel32.net
   `-    people.debian.org/~aurel32 | www.aurel32.net

      reply	other threads:[~2008-04-08  8:02 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-07 19:47 [Qemu-devel] [4164] MIPS Magnum R4000 machine Aurelien Jarno
2008-04-07 20:52 ` Paul Brook
2008-04-08  6:38 ` Hervé Poussineau
2008-04-08  8:01   ` Aurelien Jarno [this message]

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=47FB2663.7030603@aurel32.net \
    --to=aurelien@aurel32.net \
    --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).