[PATCH] DNS cache for Konqueror/KIO

Roland Harnau truthandprogress at googlemail.com
Wed Jun 25 19:03:05 BST 2008

2008/6/23, Thiago Macieira <thiago at kde.org>:
> I didn't like the indention style. Please don't put function opening
> brackets on the same line as their declaration. You also have whitespace
> errors (lines ending in whitespace).

The location of the opening brackets is maybe too much of Java style
for KDE. Attached is somewhat improved patch adhering a bit more to
kdelibs coding style and  hopefully without superfluous whitespace.

> This is also a first step in fixing the long issue of not having a
> centralised slave manager. One that would allow us to say "only one FTP
> slave per host", for instance.

"Centralised" as session- or system-wide manager? Otherwise the
Scheduler  should take care of such things.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs-DNS-cache.patch
Type: text/x-patch
Size: 14456 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080625/18aa0f1c/attachment.bin>

More information about the kde-core-devel mailing list