binary compatibility issues in kdelibs

Michael Goffioul goffioul at imec.be
Wed Jul 10 08:38:36 BST 2002


> Index: kdeprint/kmjobmanager.h
> ===================================================================
> RCS file: /home/kde/kdelibs/kdeprint/kmjobmanager.h,v
> retrieving revision 1.11
> retrieving revision 1.13
> diff -u -p -b -r1.11 -r1.13
> --- kdeprint/kmjobmanager.h     2001/11/27 16:32:44     1.11
> +++ kdeprint/kmjobmanager.h     2002/05/23 16:55:13     1.13
> @@ -71,7 +75,7 @@ protected:
>         void removeDiscardedJobs();
> 
>  protected:
> -       virtual bool listJobs(const QString& prname, JobType type);
> +       virtual bool listJobs(const QString& prname, JobType type, int limit = 0);
>         virtual bool sendCommandSystemJob(const QPtrList<KMJob>& jobs, int action, const QString& arg = QString::null);
>         bool sendCommandThreadJob(const QPtrList<KMJob>& jobs, int action, const QString& arg = QString::null);
> 
> One can also argue about this one. While the class is public it is
> commonly used as singleton, so it's unlikely that someone inherits
> from it. But still.
> 
> The same applies to KMManager.

Those 2 classes are not intended to be used by 3rd party applications
and to my knowledge, they are not used. The headers are only installed
to allow me to split KDEPrint between kdelibs and kdebase (and maybe
other locations). But those classes are only really used by KDEPrint.
So if you upgrade simultaneously kdelibs and kdebase, you shouldn't
get any problem. The only real "public" header is kprinter.h, which
I think is still BC.
Anyway, if this is a problem, I can revert the changes and remove the
added features.

Bye.
Michael.

-- 
------------------------------------------------------------------
Michael Goffioul		IMEC-DESICS-MIRA
e-mail: goffioul at imec.be	(Mixed-Signal and RF Applications)
Tel:    +32/16/28-8510		Kapeldreef, 75
Fax:    +32/16/28-1515		3001 HEVERLEE, BELGIUM
------------------------------------------------------------------




More information about the kde-core-devel mailing list