[Fontconfig-bugs] [Bug 86950] Fix or improve docs on FcConfigAppFontAddDir or FcConfigAppFontAddFile return value

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Fri Jun 26 17:06:05 PDT 2015


https://bugs.freedesktop.org/show_bug.cgi?id=86950

--- Comment #10 from Behdad Esfahbod <freedesktop at behdad.org> ---
After the changes broke many clients, I reverted the AddDir() changes and left
AddFont() in.

commit 46ec6a52d4cc447cc3ff4a13b2067ecb76c9db2e
Author: Behdad Esfahbod <behdad at behdad.org>
Date:   Fri Jun 26 17:02:13 2015 -0700

    Revert changes made to FcConfigAppFontAddDir() recently

    In 32ac7c75e8db0135ef37cf86f92d8b9be000c8bb the behavior of
    FcConfigAppFontAddFile/Dir() were changed to return false
    if not fonts were found.  While this is welldefined and useful
    for AddFile(), it's quite problematic for AddDir().  For example,
    if the directory is empty, is that a failure or success?  Worse,
    the false value from AddDir() was being propagated all the way
    to FcInit() returning false now.  This only happened upon memory
    allocation failure before, and some clients assert that FcInit()
    is successful.

    With this change, AddDir() is reverted back to what it was.
    AddFont() change (which was actually in fcdir.c) from the original
    commit is left in.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/fontconfig-bugs/attachments/20150627/b0ff7e43/attachment.html>


More information about the Fontconfig-bugs mailing list