list separator in config entries

[Andreas Pakulat]

On 08.10.07 20:02:54, Andreas Hartmetz wrote:
> 2007/10/8, Andreas Pakulat <apaku at gmx.de>:
> > Uhm, we can't change the API after 4.0, so removing the custom-separator
> > method is needed to be done _now_.
> >
> Fortunately, a random sampling of about 25 of lxr.kde.org's 170 hits
> for "readEntry" didn't turn up one single user of custom separators.
> It's totally silly, and API users seem to have realized it :)

Indeed (lxr doesn't seem to allow to search for the 3-param
read/writeEntry, stupid thing). Having run

find kdebase/ kdegraphics/ kdegames/ kdemultimedia/ kdenetwork/ kdepimlibs/ kdesdk/ kdeutils/ kdevelop kdevplatform/ -name \*.cpp -or -name \*.h | xargs grep "writeEntry([^,]\+,[^,]\+,[ ]*'"

Turned up only a few problems in these modules (and some are invalid,
like kicker stuff):

kdebase/runtime/kioslave/media/libmediacommon/notifierserviceaction.cpp:        desktopGroup.writeEntry(QString("ServiceTypes"), m_mimetypes, ',');
kdebase/workspace/kicker/applets/media/mediaapplet.cpp: c.writeEntry("ExcludedTypes", mExcludedTypesList, ';');
kdebase/workspace/kicker/applets/media/mediaapplet.cpp: c.writeEntry("ExcludedMedia", mExcludedList, ';');
kdebase/apps/konqueror/settings/filetypes/typeslistitem.cpp:    cg.writeEntry("Patterns", m_patterns, ';');
kdebase/apps/konqueror/settings/filetypes/typeslistitem.cpp:          group.writeEntry("MimeType", mimeTypeList, ';');
kdebase/apps/konqueror/settings/filetypes/typeslistitem.cpp:      group.writeEntry("MimeType", mimeTypeList, ';');
kdenetwork/lanbrowsing/kcmlisa/kcmlisa.cpp:      m_config.group("").writeEntry("PingNames",writeStuff,';');
kdesdk/kate/plugins/filetemplates/plugin/filetemplates.cpp:      cg.writeEntry( "Hidden", l, ';' );
kdesdk/kcachegrind/kcachegrind/configuration.cpp:  sourceConfig.writeEntry("Dirs", c->_generalSourceDirs, ':');

Andreas

-- 
It's lucky you're going so slowly, because you're going in the wrong direction.