Patchwork [0/2] linux-user: Fix non-UTF-8 characters

login
register
mail settings
Submitter Peter Maydell
Date Dec. 2, 2011, 7:30 p.m.
Message ID <1322854245-17590-1-git-send-email-peter.maydell@linaro.org>
Download mbox
Permalink /patch/128951/
State New
Headers show

Pull-request

git://git.linaro.org/people/pmaydell/qemu-arm.git non-utf-fixes

Comments

Peter Maydell - Dec. 2, 2011, 7:30 p.m.
These two patches fix some non-UTF-8 characters in linux-user/
source files. Other than the five files Stefan Weil recently sent
a patch to fix, these are the only remaining non-UTF-8 characters
in the source tree.

Since I don't entirely trust the email path for sending patches
to files where the diffs include non-UTF-8, I've also put the
changes up in a git repo, so you may prefer to pull from there:

The following changes since commit 1c8a881daaca6fe0646a425b0970fb3ad25f6732:

  Update version for 1.0 release (2011-12-01 14:04:21 -0600)

are available in the git repository at:
  git://git.linaro.org/people/pmaydell/qemu-arm.git non-utf-fixes

Peter Maydell (2):
      linux-user/cpu-uname.c: Convert to UTF-8
      linux-user/arm/nwfpe/fpopcode.h: Fix non-UTF-8 characters

 linux-user/arm/nwfpe/fpopcode.h |   34 +++++++++++++++++-----------------
 linux-user/cpu-uname.c          |    2 +-
 2 files changed, 18 insertions(+), 18 deletions(-)
Stefan Weil - Dec. 3, 2011, 7:55 a.m.
Am 02.12.2011 20:30, schrieb Peter Maydell:
> These two patches fix some non-UTF-8 characters in linux-user/
> source files. Other than the five files Stefan Weil recently sent
> a patch to fix, these are the only remaining non-UTF-8 characters
> in the source tree.
>
> Since I don't entirely trust the email path for sending patches
> to files where the diffs include non-UTF-8, I've also put the
> changes up in a git repo, so you may prefer to pull from there:

'git am' accepted both patches here without any problem.

> The following changes since commit 1c8a881daaca6fe0646a425b0970fb3ad25f6732:
>
>    Update version for 1.0 release (2011-12-01 14:04:21 -0600)
>
> are available in the git repository at:
>    git://git.linaro.org/people/pmaydell/qemu-arm.git non-utf-fixes
>
> Peter Maydell (2):
>        linux-user/cpu-uname.c: Convert to UTF-8
>        linux-user/arm/nwfpe/fpopcode.h: Fix non-UTF-8 characters
>
>   linux-user/arm/nwfpe/fpopcode.h |   34 +++++++++++++++++-----------------
>   linux-user/cpu-uname.c          |    2 +-
>   2 files changed, 18 insertions(+), 18 deletions(-)

Reviewed-by: Stefan Weil <sw@weilnetz.de>

I cc'ed qemu-trivial, too. Stefan, you can take both patches
into the trivial patch queue.

Thanks + regards,
Stefan Weil
Stefan Hajnoczi - Dec. 6, 2011, 10:15 a.m.
On Sat, Dec 03, 2011 at 08:55:59AM +0100, Stefan Weil wrote:
> Am 02.12.2011 20:30, schrieb Peter Maydell:
> >These two patches fix some non-UTF-8 characters in linux-user/
> >source files. Other than the five files Stefan Weil recently sent
> >a patch to fix, these are the only remaining non-UTF-8 characters
> >in the source tree.
> >
> >Since I don't entirely trust the email path for sending patches
> >to files where the diffs include non-UTF-8, I've also put the
> >changes up in a git repo, so you may prefer to pull from there:
> 
> 'git am' accepted both patches here without any problem.
> 
> >The following changes since commit 1c8a881daaca6fe0646a425b0970fb3ad25f6732:
> >
> >   Update version for 1.0 release (2011-12-01 14:04:21 -0600)
> >
> >are available in the git repository at:
> >   git://git.linaro.org/people/pmaydell/qemu-arm.git non-utf-fixes
> >
> >Peter Maydell (2):
> >       linux-user/cpu-uname.c: Convert to UTF-8
> >       linux-user/arm/nwfpe/fpopcode.h: Fix non-UTF-8 characters
> >
> >  linux-user/arm/nwfpe/fpopcode.h |   34 +++++++++++++++++-----------------
> >  linux-user/cpu-uname.c          |    2 +-
> >  2 files changed, 18 insertions(+), 18 deletions(-)
> 
> Reviewed-by: Stefan Weil <sw@weilnetz.de>
> 
> I cc'ed qemu-trivial, too. Stefan, you can take both patches
> into the trivial patch queue.

Thänk§, äppliéd to thé triviäl pätché§ -néxt tréé.

Stefan