This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

EPS import for XFig


Hello,

[Sorry for being careless and posting this recently to the patches
 list.]

There is a problem with XFig as soon as more than one EPS should
be imported as figure object.  The problem has been addressed
earlier:

  <http://article.gmane.org/gmane.os.cygwin.xfree/14757>

  <http://thread.gmane.org/gmane.os.cygwin/40818>

and finally

  <http://article.gmane.org/gmane.os.cygwin/69899>

An inspection of XFig's source code revealed that a call to
`unlink' in order to remove a temporary bitmap preview of the EPS
to include delays the actual deletion.  The bitmap preview for
subsequent EPS files would go to a temporary file always having
the same name.  As the deletion of the first preview is pending
(which means that the preview file is still there, without any
file permissions observable by Windows Explorer), subsequent
previews having the same file name can't be created and the
import into the editor view fails.

The cause for the delay of the deletion is, that there is still a
`FILE' object in memory that references that file.  The patch
forces the file to be closed as soon as it isn't needed any more.

Regards,
Franz.

2006-12-10  Franz Haeuslschmid  <lukrez@gmx.at>

	    * f_readeps.c (bitmap_from_gs): Ensures that all FILE
	      objects
	      referring to the temporary bitmap preview are
	      closed before the
	      preview file is unlinked.

--- f_readeps.c.orig  2005-10-05 11:17:05.985950000 +0200
+++ f_readeps.c	      2005-10-05 11:19:29.095325000 +0200
@@ -407,6 +407,7 @@ Boolean
         ht = pic->pic_cache->size_y;
         pcxfile = open_picfile(pixnam, &filtyp, PIPEOK,
         tmpfile);
         status = _read_pcx(pcxfile, pic);
+        fclose(pcxfile);
         /* restore width/height */
         pic->pic_cache->size_x = wid;
         pic->pic_cache->size_y = ht;





--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]