Replace KDE_open macro (and others)

Thiago Macieira thiago at kde.org
Fri Feb 13 21:39:01 GMT 2009


Christian Ehrlicher wrote:
>I would like to add them to kde_file.h. For unix it's only a wrapper
>therefore I think it's maybe worth inlining those functions - but the
>kde policy doesn't like inlining :) What do you think?

1) yes, inline

2) don't add the functions you don't need

Those functions are extremely low level. Portable code should be using 
QFile, unless it's performance critical.

In an ideal world, kde_file.h wouldn't be installed and QFile would be 
much faster...

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20090213/b5deaefa/attachment.sig>


More information about the kde-core-devel mailing list