[Patch] fix compile kdecore with msvc
Ralf Habacker
ralf.habacker at freenet.de
Sat Oct 29 22:06:16 BST 2005
Am Samstag, 29. Oktober 2005 22:26 schrieb Thiago Macieira:
> Christian Ehrlicher wrote:
> >Changes:
> >network/kresolver.cpp
> >- masked '#warning' - is gnuc-specific
>
> ok
applied
> >network/kresolver.h
> >- don't export KResolverResults to avoid compiler errors (*)
>
> Not acceptable. Don't commit this.
Christian, what kind of compiler message do you get ?
> >network/ksocketdevice.cpp
> >- don't include ksockssocketdevice in windows to avoid linker errors
>
> Understandable. I want to fix that later by other means.
>
> >- indentation fixes (tabulator -> spaces)
>
> Superfluous.
are there any astyle rules available ? It would make live much easier :-)
> >- add kde_ioctl because ioctl isn't available in windows
> >network/syssocket.h
> >- add kde_ioctl (**)
>
> Ok, makes sense.
applied
> >* there is imho no need to export KResolverResults because nobody
> >derivates a class from KResolverResults. And if you really want to
> >export this class, you have to implement all functions QList<T> defines
> >because otherwise when someone wants to use a function you forgot to
> >implement he will get a linker error.
>
> Your argument doesn't make sense. Functions that were not overridden will
> be inherited from QList<T>.
Ralf
More information about the kde-core-devel
mailing list