Re: [PATCH] src/fccache.c: Fix define for HAVE_POSIX_FADVISE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks for catching this up. this seems somehow wrongly removed. fixed in git.


On Fri, Jan 25, 2019 at 11:36 AM Robert Yang <liezhi.yang@xxxxxxxxxxxxx> wrote:
>
> Otherwise, there would be build errors in the following 2 cases:
> * define HAVE_POSIX_FADVISE
> Or:
> * undef HAVE_POSIX_FADVISE
>
> Signed-off-by: Robert Yang <liezhi.yang@xxxxxxxxxxxxx>
> ---
>  src/fccache.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/fccache.c b/src/fccache.c
> index 3352a66..ac46ee8 100644
> --- a/src/fccache.c
> +++ b/src/fccache.c
> @@ -976,7 +976,7 @@ FcDirCacheMapFd (FcConfig *config, int fd, struct stat *fd_stat, struct stat *di
>      {
>  #if defined(HAVE_MMAP) || defined(__CYGWIN__)
>         cache = mmap (0, fd_stat->st_size, PROT_READ, MAP_SHARED, fd, 0);
> -#if (HAVE_POSIX_FADVISE) && defined(POSIX_FADV_WILLNEED)
> +#if defined(HAVE_POSIX_FADVISE) && defined(POSIX_FADV_WILLNEED)
>         posix_fadvise (fd, 0, fd_stat->st_size, POSIX_FADV_WILLNEED);
>  #endif
>         if (cache == MAP_FAILED)
> --
> 2.7.4
>
> _______________________________________________
> Fontconfig mailing list
> Fontconfig@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/fontconfig



--
Akira TAGOH
_______________________________________________
Fontconfig mailing list
Fontconfig@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/fontconfig




[Index of Archives]     [Fedora Fonts]     [Fedora Users]     [Fedora Cloud]     [Kernel]     [Fedora Packaging]     [Fedora Desktop]     [PAM]     [Gimp Graphics Editor]     [Yosemite News]

  Powered by Linux