Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-28460

missing platform implementation for QFontDatabase::removeApplicationFont()

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Invalid
    • Icon: P2: Important P2: Important
    • None
    • 5.0.0 RC 1
    • GUI: Font handling
    • None

      1. missed a bridge call to QPlatformFontDatabase::releaseHandle() from QFontDatabase::removeApplicationFont().

      2. missed the implementation for fontconfig at least
      in 4.8 code, you could try "git grep removeApplicationFont" in src/gui:

      ......
text/qfontdatabase_mac.cpp:bool QFontDatabase::removeApplicationFont(int handle)
text/qfontdatabase_mac.cpp:        if(!removeApplicationFont(i))
text/qfontdatabase_qpa.cpp:bool QFontDatabase::removeApplicationFont(int handle)
text/qfontdatabase_qws.cpp:bool QFontDatabase::removeApplicationFont(int handle)
text/qfontdatabase_s60.cpp:bool QFontDatabase::removeApplicationFont(int handle)
text/qfontdatabase_s60.cpp:        if (!removeApplicationFont(i))
text/qfontdatabase_win.cpp:bool QFontDatabase::removeApplicationFont(int handle)
text/qfontdatabase_win.cpp:        if (!removeApplicationFont(i))
text/qfontdatabase_x11.cpp:bool QFontDatabase::removeApplicationFont(int handle)

      in 5.0, "git grep QPlatformFontDatabase":

      ......
platformsupport/fontdatabases/basic/qbasicfontdatabase.cpp:void QBasicFontDatabase::releaseHandle(void *handle)
platformsupport/fontdatabases/basic/qbasicfontdatabase_p.h:    void releaseHandle(void *handle);
platformsupport/fontdatabases/mac/qcoretextfontdatabase.mm:void QCoreTextFontDatabase::releaseHandle(void *handle)
platformsupport/fontdatabases/mac/qcoretextfontdatabase_p.h:    void releaseHandle(void *handle);
plugins/platforms/windows/qwindowsfontdatabase.cpp:void QWindowsFontDatabase::releaseHandle(void *handle)
plugins/platforms/windows/qwindowsfontdatabase.h:    virtual void releaseHandle(void *handle);

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            esabraha Eskil Abrahamsen Blomfeldt
            liaqi Liang Qi
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes