all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Joseph Garvin <k04jg02@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Alan Third <alan@idiocy.org>, emacs-devel@gnu.org
Subject: Re: Linking to ImageMagick by default
Date: Wed, 19 Dec 2018 10:45:45 -0600	[thread overview]
Message-ID: <CAH=MJ4ZpAg0qsNduDdWnCY2bJmEaSUn18OyVg2Dzd4hNgjxDrA@mail.gmail.com> (raw)
In-Reply-To: <83efadcw3j.fsf@gnu.org>

[-- Attachment #1: Type: text/plain, Size: 1568 bytes --]

What does Mozilla use for Firefox? They should have an even more serious
security outlook.

On Wed, Dec 19, 2018, 10:36 AM Eli Zaretskii <eliz@gnu.org wrote:

> > Date: Wed, 19 Dec 2018 16:03:08 +0000
> > From: Alan Third <alan@idiocy.org>
> >
> > I want to strip out the properties that I’m going to use at display
> > time, so that I can use the same cached image for different output
> > sizes, while leaving other properties that I don’t know about in
> > place. So I’d like to store the above as
> >
> >     '(image :file "image.png" :format png)
> >
> > however I’m not sure how to go about filtering the list in C. I tried
> > a few variations on this, but they all seg fault
> >
> > static Lisp_Object
> > get_cache_spec (Lisp_Object spec)
> > {
> >   Lisp_Object cache_spec, tail;
> >
> >   cache_spec = Qnil;
> >
> >   for (tail = XCDR (spec);
> >        CONSP (tail) && CONSP (XCDR (tail));
> >        tail = XCDR (XCDR (tail)))
> >     {
> >       if (!EQ (XCAR (tail), Qwidth)
> >           && !EQ (XCAR (tail), QCheight)
> >           && !EQ (XCAR (tail), QCmax_width)
> >           && !EQ (XCAR (tail), QCmax_height)
> >           && !EQ (XCAR (tail), QCscale)
> >           && !EQ (XCAR (tail), QCmargin)
> >           && !EQ (XCAR (tail), QCascent)
> >           && !EQ (XCAR (tail), QCrelief))
> >         cache_spec = list3 (XCAR (tail), XCAR (XCDR (tail)), cache_spec);
> >     }
> >   cache_spec = list2 (XCAR (spec), cache_spec);
>
> I think you should use Fcons instead of list3 or list2.
>
>

[-- Attachment #2: Type: text/html, Size: 2211 bytes --]

  reply	other threads:[~2018-12-19 16:45 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-12-02  1:33 Linking to ImageMagick by default Glenn Morris
2018-12-02 18:15 ` Paul Eggert
2018-12-05 13:30 ` Lars Ingebrigtsen
2018-12-05 15:28   ` Stefan Monnier
2018-12-06 11:06     ` Lars Ingebrigtsen
2018-12-05 17:24   ` Glenn Morris
2018-12-05 17:27     ` Lars Ingebrigtsen
2018-12-05 18:27     ` Daniel Pittman
2018-12-05 18:38       ` Lars Ingebrigtsen
2018-12-05 19:31     ` joakim
2018-12-05 22:39   ` Alan Third
2018-12-08 18:38     ` Alan Third
2018-12-08 21:24       ` Paul Eggert
2018-12-10 22:09         ` Alan Third
2018-12-19 16:03           ` Alan Third
2018-12-19 16:36             ` Eli Zaretskii
2018-12-19 16:45               ` Joseph Garvin [this message]
2018-12-27 15:06                 ` Alan Third
2018-12-27 13:11               ` Alan Third
2018-12-27 16:05                 ` Eli Zaretskii
2018-12-27 20:37                   ` Juri Linkov
2018-12-28  8:12                 ` Eli Zaretskii
2018-12-28 21:21                   ` Alan Third
2018-12-29  6:56                     ` Eli Zaretskii
2018-12-29 21:31                       ` Juri Linkov
2018-12-30 12:47                       ` Alan Third
2019-01-01 21:47                         ` [PATCH] Add native image scaling Alan Third
2019-01-02 16:11                           ` Eli Zaretskii
2019-01-02 21:12                             ` [PATCH v2] Add native image scaling (bug#33587) Alan Third
2019-01-04 14:31                               ` Eli Zaretskii
2019-01-04 19:09                                 ` Alan Third
2019-01-04 20:21                                   ` Eli Zaretskii
2019-01-04 22:45                                     ` Alan Third
2019-01-10 19:42                                       ` Alan Third
2019-01-10 19:50                                         ` Eli Zaretskii
2019-01-10 23:40                                         ` bug#33587: " Paul Eggert
2019-01-10 23:40                                         ` Paul Eggert
2019-05-14  6:15                                           ` bug#33587: [PROPOSED] Default to disabling ImageMagick Paul Eggert
2019-01-06 16:26                                   ` [PATCH v2] Add native image scaling (bug#33587) Eli Zaretskii
2019-01-05 21:30                               ` Track mouse drags over an image (was: Add native image scaling) Roland Winkler
2019-01-08 18:20                                 ` Track mouse drags over an image Stefan Monnier
2019-01-11  4:55                                   ` Roland Winkler
2019-01-11 16:23                                     ` Stefan Monnier
2018-12-09 11:34       ` Linking to ImageMagick by default Elias Mårtenson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAH=MJ4ZpAg0qsNduDdWnCY2bJmEaSUn18OyVg2Dzd4hNgjxDrA@mail.gmail.com' \
    --to=k04jg02@gmail.com \
    --cc=alan@idiocy.org \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.