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

List:       kde-pim
Subject:    [Kde-pim] bad use of fTray in pilotDaemon.cc
From:       Mark Hollomon <mhh () mindspring ! com>
Date:       2001-11-09 19:56:36
[Download RAW message or body]

This bit of code is suspect since fTray may not be defined:

	if (KPilotConfig::getConfig().resetGroup().getSyncFiles())
	{
		fP->addAction(new FileInstallAction(fPilotLink,
				fTray->installDir(), fTray->installFiles()));
	}

If the daemon isn't being shown in the tray, fTray will be NULL.
The method guards some earlier code with "if (fTray)". I can't recommend
a patch because I don't know (yet) where else the info that the
FileInstallAction constructor requires may be located.

-- 
Mark Hollomon
_______________________________________________
kde-pim mailing list
kde-pim@mail.kde.org
http://mail.kde.org/mailman/listinfo/kde-pim

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

Configure | About | News | Add a list | Sponsored by KoreLogic