Mercurial > sdl-ios-xcode
view SDL.spec.in @ 3539:f2846bf19360
Fixed bug #896
John Popplewell 2009-12-08 23:05:50 PST
Originally reported by AKFoerster on the mailing list.
Error decoding UTF8 Russian text to UTF-16LE on Windows, but specifically on
platforms without iconv support (the default on Windows).
Valid UTF8 characters are flagged as being overlong and then substituted by the
UNKNOWN_UNICODE character.
After studying the testiconv.c example program, reading the RFCs and putting
some printf statements in SDL_iconv.c the problem is in a test for 'Maximum
overlong sequences', specifically 4.2.1, which is carried out by the following
code:
} else if ( p[0] >= 0xC0 ) {
if ( (p[0] & 0xE0) != 0xC0 ) {
/* Skip illegal sequences
return SDL_ICONV_EILSEQ;
*/
ch = UNKNOWN_UNICODE;
} else {
if ( (p[0] & 0xCE) == 0xC0 ) { <<<<<<<< here
overlong = SDL_TRUE;
}
ch = (Uint32)(p[0] & 0x1F);
left = 1;
}
} else {
Here is the 2-byte encoding of a character in range 00000080 - 000007FF
110xxxxx 10xxxxxx
The line in question is supposed to be checking for an overlong sequence which
would be less than
11000001 10111111
which should be represented as a single byte.
BUT, the mask value (0xCE) is wrong, it isn't checking the top-most bit:
11000001 value
11001110 mask (incorrect)
^
and should be (0xDE):
11000001 value
11011110 mask (correct)
making the above code:
} else if ( p[0] >= 0xC0 ) {
if ( (p[0] & 0xE0) != 0xC0 ) {
/* Skip illegal sequences
return SDL_ICONV_EILSEQ;
*/
ch = UNKNOWN_UNICODE;
} else {
if ( (p[0] & 0xDE) == 0xC0 ) { <<<<<<<< here
overlong = SDL_TRUE;
}
ch = (Uint32)(p[0] & 0x1F);
left = 1;
}
} else {
I can supply a test program and/or a patch if required,
best regards,
John Popplewell
author | Sam Lantinga <slouken@libsdl.org> |
---|---|
date | Fri, 11 Dec 2009 08:03:43 +0000 |
parents | 62debeaac5e9 |
children | 17376128fb56 |
line wrap: on
line source
Summary: Simple DirectMedia Layer Name: SDL Version: @SDL_VERSION@ Release: 1 Source: http://www.libsdl.org/release/%{name}-%{version}.tar.gz URL: http://www.libsdl.org/ License: LGPL Group: System Environment/Libraries BuildRoot: %{_tmppath}/%{name}-%{version}-buildroot Prefix: %{_prefix} %ifos linux Provides: libSDL-1.2.so.0 %endif %define __defattr %defattr(-,root,root) %define __soext so %description This is the Simple DirectMedia Layer, a generic API that provides low level access to audio, keyboard, mouse, and display framebuffer across multiple platforms. %package devel Summary: Libraries, includes and more to develop SDL applications. Group: Development/Libraries Requires: %{name} = %{version} %description devel This is the Simple DirectMedia Layer, a generic API that provides low level access to audio, keyboard, mouse, and display framebuffer across multiple platforms. This is the libraries, include files and other resources you can use to develop SDL applications. %prep %setup -q %build %ifos linux CFLAGS="$RPM_OPT_FLAGS" ./configure --prefix=%{prefix} --disable-video-aalib --disable-video-directfb --disable-video-ggi --disable-video-svga %else %configure %endif make %install rm -rf $RPM_BUILD_ROOT %ifos linux make install prefix=$RPM_BUILD_ROOT%{prefix} \ bindir=$RPM_BUILD_ROOT%{_bindir} \ libdir=$RPM_BUILD_ROOT%{_libdir} \ includedir=$RPM_BUILD_ROOT%{_includedir} \ datadir=$RPM_BUILD_ROOT%{_datadir} \ mandir=$RPM_BUILD_ROOT%{_mandir} ln -s libSDL-1.2.so.0 $RPM_BUILD_ROOT%{_libdir}/libSDL-1.1.so.0 %else %makeinstall %endif %clean rm -rf $RPM_BUILD_ROOT %files %{__defattr} %doc README-SDL.txt COPYING CREDITS BUGS %{_libdir}/lib*.%{__soext}.* %files devel %{__defattr} %doc README README-SDL.txt COPYING CREDITS BUGS WhatsNew docs.html %doc docs/index.html docs/html %{_bindir}/*-config %{_libdir}/lib*.a %{_libdir}/lib*.la %{_libdir}/lib*.%{__soext} %dir %{_includedir}/SDL %{_includedir}/SDL/*.h %{_libdir}/pkgconfig/sdl.pc %{_datadir}/aclocal/* %{_mandir}/man3/* %changelog * Tue May 16 2006 Sam Lantinga <slouken@libsdl.org> - Removed support for Darwin, due to build problems on ps2linux * Mon Jan 03 2004 Anders Bjorklund <afb@algonet.se> - Added support for Darwin, updated spec file * Wed Jan 19 2000 Sam Lantinga <slouken@libsdl.org> - Re-integrated spec file into SDL distribution - 'name' and 'version' come from configure - Some of the documentation is devel specific - Removed SMP support from %build - it doesn't work with libtool anyway * Tue Jan 18 2000 Hakan Tandogan <hakan@iconsult.com> - Hacked Mandrake sdl spec to build 1.1 * Sun Dec 19 1999 John Buswell <johnb@mandrakesoft.com> - Build Release * Sat Dec 18 1999 John Buswell <johnb@mandrakesoft.com> - Add symlink for libSDL-1.0.so.0 required by sdlbomber - Added docs * Thu Dec 09 1999 Lenny Cartier <lenny@mandrakesoft.com> - v 1.0.0 * Mon Nov 1 1999 Chmouel Boudjnah <chmouel@mandrakesoft.com> - First spec file for Mandrake distribution. # end of file