From kde-pim  Sun Jan 06 17:36:16 2008
From: Tobias Koenig <tokoe () kde ! org>
Date: Sun, 06 Jan 2008 17:36:16 +0000
To: kde-pim
Subject: Re: [Kde-pim] [patch] vcard parsing in KDE3.5
Message-Id: <20080106173616.GB3636 () ghostdog ! localnet>
X-MARC-Message: https://marc.info/?l=kde-pim&m=119964080531042
MIME-Version: 1
Content-Type: multipart/mixed; boundary="--===============1894643306=="


--===============1894643306==
Content-Type: multipart/signed; micalg=pgp-sha1;
	protocol="application/pgp-signature"; boundary="8GpibOaaTibBMecb"
Content-Disposition: inline


--8GpibOaaTibBMecb
Content-Type: text/plain; charset=iso-8859-15
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Jan 05, 2008 at 01:52:16PM +0100, Martin Koller wrote:
> Hi all,
Hej Martin,

> attached is a patch that corrects the vcard parsing bug (kdelibs/kabc) de=
aling=20
> with different charsets in vcards.
> It solves e.g. https://bugs.kde.org/show_bug.cgi?id=3D98790
> This is mainly a backport from KDE4 where it's already solved.
Well, in KDE4 the input for the vCard parser is a QByteArray, in other
words plain data without previous convertion. In KDE3 the vCard parser
expects a QString, so you always need at least one convertion from the
raw data (QCString) to QString. From your patch it looks like to always
use latin1. Are you sure that doesn't remove any information from the
input data?

The right fix would be to use QCString as input data for vCard parser,
however that's not possible because it's SIC/BIC. (I know, bad design
in 3.x cycle...)

So if you are _really, really_ sure that your patch doesn't break _any_
other usage of the parser in KDE3, I'm fine with a commit, otherwise I
don't see a chance to fix it.

Ciao,
Tobias
--=20
Separate politics from religion and economy!
The Council of the European Union is an undemocratic and illegal institutio=
n!

--8GpibOaaTibBMecb
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFHgRGQSvFUKpY6VLARAq7NAJ9xJmXNeHSEJ1ZCp2i0ZQNwcmwdzgCfUS/t
fyN3Q7nkwMJlZUutbqjdJZA=
=eLMD
-----END PGP SIGNATURE-----

--8GpibOaaTibBMecb--

--===============1894643306==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
--===============1894643306==--