unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: 23.0.60; gs.el - This code is experimental.  Don't use it.
@ 2008-12-09 23:51 Chong Yidong
  2009-01-10  5:10 ` Leo
  0 siblings, 1 reply; 7+ messages in thread
From: Chong Yidong @ 2008-12-09 23:51 UTC (permalink / raw)
  To: emacs-devel; +Cc: 1524, Leo

> I really have no idea what gs.el does. The code looks sketchy leading me
> to believe the comment in its header actually is true.
>
>   This code is experimental.  Don't use it.

Looks like cruft to me too.

Does anyone object to removing this file?




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: 23.0.60; gs.el - This code is experimental.  Don't use it.
  2008-12-09 23:51 23.0.60; gs.el - This code is experimental. Don't use it Chong Yidong
@ 2009-01-10  5:10 ` Leo
  2009-01-10 13:22   ` Chong Yidong
  0 siblings, 1 reply; 7+ messages in thread
From: Leo @ 2009-01-10  5:10 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 1524, emacs-devel

On 2008-12-09 23:51 +0000, Chong Yidong wrote:
>> I really have no idea what gs.el does. The code looks sketchy leading me
>> to believe the comment in its header actually is true.
>>
>>   This code is experimental.  Don't use it.
>
> Looks like cruft to me too.
>
> Does anyone object to removing this file?

I think it is safe to remove (info "(elisp)PostScript Images") and gs.el
now that over a month has passed.

Best,

Leo




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: 23.0.60; gs.el - This code is experimental.  Don't use it.
  2009-01-10  5:10 ` Leo
@ 2009-01-10 13:22   ` Chong Yidong
  2009-01-11  3:51     ` Leo
  0 siblings, 1 reply; 7+ messages in thread
From: Chong Yidong @ 2009-01-10 13:22 UTC (permalink / raw)
  To: Leo; +Cc: 1524, emacs-devel

Leo <sdl.web@gmail.com> writes:

> I think it is safe to remove (info "(elisp)PostScript Images") and
> gs.el now that over a month has passed.

It's used in image.c.  We should probably try to fix it if it doesn't
work properly.




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: 23.0.60; gs.el - This code is experimental.  Don't use it.
  2009-01-10 13:22   ` Chong Yidong
@ 2009-01-11  3:51     ` Leo
  2009-01-11 14:57       ` Stefan Monnier
  0 siblings, 1 reply; 7+ messages in thread
From: Leo @ 2009-01-11  3:51 UTC (permalink / raw)
  To: Chong Yidong; +Cc: 1524, emacs-devel

On 2009-01-10 13:22 +0000, Chong Yidong wrote:
> Leo <sdl.web@gmail.com> writes:
>
>> I think it is safe to remove (info "(elisp)PostScript Images") and
>> gs.el now that over a month has passed.
>
> It's used in image.c.  We should probably try to fix it if it doesn't
> work properly.

I don't use it so I don't know whether it has bugs. I suspect it
has. But I'm mostly concerned by the comment in the source which advise
users not to use it.

Best,

Leo




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: 23.0.60; gs.el - This code is experimental.  Don't use it.
  2009-01-11  3:51     ` Leo
@ 2009-01-11 14:57       ` Stefan Monnier
  2009-02-04 15:56         ` Leo
  2009-02-04 16:16         ` Leo
  0 siblings, 2 replies; 7+ messages in thread
From: Stefan Monnier @ 2009-01-11 14:57 UTC (permalink / raw)
  To: Leo; +Cc: Chong Yidong, kastrup, 1524, emacs-devel

>>> I think it is safe to remove (info "(elisp)PostScript Images") and
>>> gs.el now that over a month has passed.
>> It's used in image.c.  We should probably try to fix it if it doesn't
>> work properly.
> I don't use it so I don't know whether it has bugs. I suspect it
> has. But I'm mostly concerned by the comment in the source which advise
> users not to use it.

IIRC David Kastrup would know how broken it is.


        Stefan




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: 23.0.60; gs.el - This code is experimental. Don't use it.
  2009-01-11 14:57       ` Stefan Monnier
@ 2009-02-04 15:56         ` Leo
  2009-02-04 16:16         ` Leo
  1 sibling, 0 replies; 7+ messages in thread
From: Leo @ 2009-02-04 15:56 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Chong Yidong, 1524, kastrup, Leo, emacs-devel

2009/1/11 Stefan Monnier <monnier@iro.umontreal.ca>:
>>>> I think it is safe to remove (info "(elisp)PostScript Images") and
>>>> gs.el now that over a month has passed.
>>> It's used in image.c.  We should probably try to fix it if it doesn't
>>> work properly.
>> I don't use it so I don't know whether it has bugs. I suspect it
>> has. But I'm mostly concerned by the comment in the source which advise
>> users not to use it.
>
> IIRC David Kastrup would know how broken it is.

Ping?

>        Stefan
>




^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: 23.0.60; gs.el - This code is experimental.  Don't use it.
  2009-01-11 14:57       ` Stefan Monnier
  2009-02-04 15:56         ` Leo
@ 2009-02-04 16:16         ` Leo
  1 sibling, 0 replies; 7+ messages in thread
From: Leo @ 2009-02-04 16:16 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: Chong Yidong, dak, 1524, emacs-devel

On 2009-01-11 14:57 +0000, Stefan Monnier wrote:
[...]
> IIRC David Kastrup would know how broken it is.

Here is David's response.

------------------------------------------------
Leo <sdl.web@googlemail.com> writes:

> 2009/1/11 Stefan Monnier <monnier@iro.umontreal.ca>:
>>>>> I think it is safe to remove (info "(elisp)PostScript Images") and
>>>>> gs.el now that over a month has passed.
>>>> It's used in image.c.  We should probably try to fix it if it
>doesn't
>>>> work properly.
>>> I don't use it so I don't know whether it has bugs. I suspect it
>>> has. But I'm mostly concerned by the comment in the source which
>advise
>>> users not to use it.
>>
>> IIRC David Kastrup would know how broken it is.
>
> Stefan's email sent an email address of yours which is no longer in
> use. Could you comment this in emacs-devel mailing list?

Pass it on if you want to.  There are a lot of problems due to
Ghostscript rendering directly to Pixmaps.  Every image is rendered with
its own Ghostscript process, but the parameter passing requires X
properties as a global resource.  This is done once per Ghostscript
session and includes the display dimensions, so it can't really be
extended to multiple image rendering without further Ghostscript API
changes.

The display code triggering the display is rather asynchronous: I've not
been able figuring out how it is rooted in the bowels of the display
engine.

In practice, it worked more or less for a single image rendering per
Emacs session.  It doesn't scale at all.  Serious work both in
Ghostscript as well as in Emacs would be required to make this a smooth
working feature, and I see no candidate for either.
------------------------------------------------




^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2009-02-04 16:16 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-12-09 23:51 23.0.60; gs.el - This code is experimental. Don't use it Chong Yidong
2009-01-10  5:10 ` Leo
2009-01-10 13:22   ` Chong Yidong
2009-01-11  3:51     ` Leo
2009-01-11 14:57       ` Stefan Monnier
2009-02-04 15:56         ` Leo
2009-02-04 16:16         ` Leo

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).