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

missing platform implementation for QFontDatabase::removeApplicationFont()

    XMLWordPrintable

Details

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

    Description

      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);
      

      Attachments

        Issue Links

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

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes