[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdeutils/karm
From:       Antonio Larrosa <larrosa () suse ! de>
Date:       2000-07-31 22:16:13
[Download RAW message or body]

"Jesper K. Pedersen" escribió:
> 
> CVS by antlarr <kde-cvs@kde.org> writes:
> 
> | kdeutils/karm top.cpp,1.45,1.46
> | Author: antlarr
> | CVSROOT: /home/kde
> | Mon Jul 31 16:44:08 UTC 2000
> | Update of /home/kde/kdeutils/karm
> | cvs.kde.org hosted by sourceforge.net
> |
> | Modified Files:
> |       top.cpp
> | Log Message:
> | I "love" to revert recently applied bugfixes, but I'd prefer to find
> | a better fix for this.
> I'm a bit in doubt now. Could you please elaborate on what need to be
> changed.
> 

Well, that file uses extensively QString::fromLatin1 to convert char *
to QString. for example: createGUI( QString::fromLatin1("karmui.rc") );

If you just leave createGUI( "karmui.rc" ); (as in every other place in
KDE), you get compiler errors because there seems to be no "implicit"
QString constructor that takes just a char *  (!?!?)

So it seems to me that there's some kind of header files problem.

Anyway, if you're mantaining this app, I'll just leave the problem for
you (Ok, it wasn't a bugfix, but it was a "maintenance fix" :) )

Greetings,

--
Antonio Larrosa Jimenez
KDE Core developer
antlarr@arrakis.es        larrosa@kde.org
http://www.arrakis.es/~rlarrosa
KDE - The development framework of the future, today.

[prev in list] [next in list] [prev in thread] [next in thread]