* [PATCH 0/5] [RFC] fixes for w32 and NetPBM
@ 2013-10-25 15:35 Claudio Bley
2013-10-25 18:45 ` Eli Zaretskii
0 siblings, 1 reply; 5+ messages in thread
From: Claudio Bley @ 2013-10-25 15:35 UTC (permalink / raw)
To: emacs-devel
Hi.
Here are a few patches that have piled up in my local branch over the
last few months.
The first three patches support to use a recent JPEG, PNG and TIFF
library, respectively. I'm currently using libpng 1.6.6, jpeglib
version 9 and libtiff 4.0.3.
The last two patches (could perhaps be integrated into one?) fix
parsing of NetPBM headers.
Any comments would be welcome, thanks.
Claudio Bley (5):
w32: define new jpeglib-version var in order to load the correct DLL
at runtime
w32: add support for recent PNG library version >= 1.5
w32: add support for default library names for libtiff
Fix parsing of NetPBM file comments
Fix file magic for pbm files with comments
lisp/image.el | 5 +++-
lisp/term/w32-win.el | 22 +++++++++++-------
src/image.c | 65 +++++++++++++++++++++++++++++++++++-----------------
3 files changed, 62 insertions(+), 30 deletions(-)
--
1.8.4.msysgit.0
--
Claudio
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/5] [RFC] fixes for w32 and NetPBM
2013-10-25 15:35 [PATCH 0/5] [RFC] fixes for w32 and NetPBM Claudio Bley
@ 2013-10-25 18:45 ` Eli Zaretskii
2013-10-29 7:02 ` Claudio Bley
0 siblings, 1 reply; 5+ messages in thread
From: Eli Zaretskii @ 2013-10-25 18:45 UTC (permalink / raw)
To: Claudio Bley; +Cc: emacs-devel
> From: claudio.bley@gmail.com (Claudio Bley)
> Date: Fri, 25 Oct 2013 17:35:18 +0200
>
> Here are a few patches that have piled up in my local branch over the
> last few months.
Thanks.
> w32: define new jpeglib-version var in order to load the correct DLL
> at runtime
Are the newer DLLs binary incompatible with the older ones? If not,
I'd prefer not to introduce any version checking, since this increases
the probability of the user's DLL being rejected by Emacs for no good
reason.
If there are indeed incompatibilities, can you point to the
documentation that describes them?
> w32: add support for recent PNG library version >= 1.5
Likewise here: do versions >= 1.5 indeed again broke binary
compatibility with earlier versions?
> w32: add support for default library names for libtiff
Where did you find these newer ports of libtiff? Maybe we should
amend the instructions in nt/INSTALL and nt/README.W32.
> Fix file magic for pbm files with comments
I don't think I've seen this part. Did you send it?
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/5] [RFC] fixes for w32 and NetPBM
2013-10-25 18:45 ` Eli Zaretskii
@ 2013-10-29 7:02 ` Claudio Bley
2013-11-01 9:32 ` Claudio Bley
0 siblings, 1 reply; 5+ messages in thread
From: Claudio Bley @ 2013-10-29 7:02 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: emacs-devel
At Fri, 25 Oct 2013 21:45:19 +0300,
Eli Zaretskii wrote:
>
> > From: claudio.bley@gmail.com (Claudio Bley)
> > Date: Fri, 25 Oct 2013 17:35:18 +0200
> >
> > Here are a few patches that have piled up in my local branch over the
> > last few months.
>
> Thanks.
>
> > w32: define new jpeglib-version var in order to load the correct DLL
> > at runtime
>
> Are the newer DLLs binary incompatible with the older ones? If not,
> I'd prefer not to introduce any version checking, since this increases
> the probability of the user's DLL being rejected by Emacs for no good
> reason.
Yes, it seems all releases since version 7 are incompatible with the
former ones.
> If there are indeed incompatibilities, can you point to the
> documentation that describes them?
I don't have a link to official documentation at hand (the jpeglib
website is a bit sparse), so I'm quoting wikipedia
(https://en.wikipedia.org/wiki/Jpeglib):
,----
| There are currently two conditions which trigger a major version bump
| of the JPEG library:
|
| 1. New types of files can be created which are not compatible with prior releases, and
| 2. Changes in the master records which break the binary compatibility (usually addition of entries for new functions).
|
| In all major upgrades since v6 both conditions 1. and 2. came
| together, while the source compatibility (API) could largely be
| retained.
`----
> > w32: add support for default library names for libtiff
>
> Where did you find these newer ports of libtiff? Maybe we should
> amend the instructions in nt/INSTALL and nt/README.W32.
I'm using the latest officially released version from
http://www.remotesensing.org/libtiff/.
> > Fix file magic for pbm files with comments
>
> I don't think I've seen this part. Did you send it?
Somehow this part didn't make it. I'll re-send.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/5] [RFC] fixes for w32 and NetPBM
2013-10-29 7:02 ` Claudio Bley
@ 2013-11-01 9:32 ` Claudio Bley
2013-11-01 9:49 ` Eli Zaretskii
0 siblings, 1 reply; 5+ messages in thread
From: Claudio Bley @ 2013-11-01 9:32 UTC (permalink / raw)
To: emacs-devel
version 2 has been posted here:
http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00931.html
--
Claudio
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/5] [RFC] fixes for w32 and NetPBM
2013-11-01 9:32 ` Claudio Bley
@ 2013-11-01 9:49 ` Eli Zaretskii
0 siblings, 0 replies; 5+ messages in thread
From: Eli Zaretskii @ 2013-11-01 9:49 UTC (permalink / raw)
To: Claudio Bley; +Cc: emacs-devel
> From: Claudio Bley <claudio.bley@gmail.com>
> Date: Fri, 01 Nov 2013 10:32:23 +0100
>
> version 2 has been posted here:
>
> http://lists.gnu.org/archive/html/emacs-devel/2013-10/msg00931.html
And I announced the commit here:
http://lists.gnu.org/archive/html/emacs-devel/2013-11/msg00008.html
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-11-01 9:49 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-10-25 15:35 [PATCH 0/5] [RFC] fixes for w32 and NetPBM Claudio Bley
2013-10-25 18:45 ` Eli Zaretskii
2013-10-29 7:02 ` Claudio Bley
2013-11-01 9:32 ` Claudio Bley
2013-11-01 9:49 ` Eli Zaretskii
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.