[Fontconfig] [PATCH 0/1] Add sysroot option to fc-cache and fc-cat
Laurentiu Palcu
laurentiu.palcu at intel.com
Tue Jan 29 04:11:56 PST 2013
Hi Akira,
On 01/29/2013 03:22 AM, Akira TAGOH wrote:
> Please make the modified code MT-safety and good to attach the patch as
> the file or into the bugzilla next time.
Attached a v2 patch to bug #59456.
Thanks,
Laurentiu
>
> Thanks,
>
>
> On Mon, Jan 28, 2013 at 9:20 PM, Laurentiu Palcu
> <laurentiu.palcu at intel.com <mailto:laurentiu.palcu at intel.com>> wrote:
>
> Hi all,
>
>
> This patch will add a --sysroot option to fc-cache and fc-cat tools.
> More
> explanations have been included in the commit message of the patch
> itself.
>
> Please review the patch but bear in mind that I only took contact
> with fontconfig
> source code approx a week ago. So, even though I tried my best to
> respect the
> coding convention and existing API, there are things I might have
> missed.
>
> I tested the patch both locally and on target. That is, after I
> deployed the image,
> I re-run fc-cache (without sysroot) and saw that the generated cache
> was identical.
>
> I also ran the tools under valgrind and, apparently, there were no
> leaks introduced.
>
> I sent the changes within only one patch as it adds only one logical
> change. But, if
> you guys have other rules (one patch per file, etc.), let me know
> and I'll resend.
>
> That being said, please review/test the changes yourselves and let
> me know your
> concerns/thoughts/etc.
>
> Thanks,
> Laurentiu
>
> Laurentiu Palcu (1):
> Add sysroot option to fc-cache and fc-cat
>
> fc-cache/fc-cache.c | 57 ++++++++++++++++++-----
> fc-cat/fc-cat.c | 54 +++++++++++++++++----
> fontconfig/fontconfig.h | 6 +++
> src/fccache.c | 119
> ++++++++++++++++++++++++++++++++++++++++-------
> src/fccfg.c | 32 +++++++++++++
> src/fcfreetype.c | 4 ++
> src/fcstr.c | 18 ++++++-
> 7 files changed, 251 insertions(+), 39 deletions(-)
>
> --
> 1.7.9.5
>
> _______________________________________________
> Fontconfig mailing list
> Fontconfig at lists.freedesktop.org
> <mailto:Fontconfig at lists.freedesktop.org>
> http://lists.freedesktop.org/mailman/listinfo/fontconfig
>
>
>
>
> --
> Akira TAGOH
More information about the Fontconfig
mailing list