FLTK logo

STR #3037

FLTK matrix user chat room
(using Element browser app)   FLTK gitter user chat room   GitHub FLTK Project   FLTK News RSS Feed  
  FLTK Apps      FLTK Library      Forums      Links     Login 
 Home  |  Articles & FAQs  |  Bugs & Features  |  Documentation  |  Download  |  Screenshots  ]
 

Return to Bugs & Features | Roadmap 1.3 | SVN ⇄ GIT ]

STR #3037

Application:FLTK Library
Status:1 - Closed w/Resolution
Priority:2 - Low, e.g. a documentation error or undocumented side-effect
Scope:3 - Applies to all machines and operating systems
Subsystem:Image Support
Summary:multiple deprecations in built-in zlib
Version:1.3-current
Created By:AnyCPU
Assigned To:AlbrechtS
Fix Version:1.3.4 (SVN: v10624)
Update Notification:

Receive EMails Don't Receive EMails

Trouble Report Files:

No files


Trouble Report Comments:


Name/Time/Date Text  
 
#1 AnyCPU
14:00 Jan 25, 2014
has been explicitly marked deprecated:

lseek, close, write

If possible should be ISO's: _lseek, _close, _write
 
 
#2 ianmacarthur
01:37 May 01, 2014
I suspect this one is a bit "won't fix" as we want to track the imported libs.

I'd imagine this might "fix" itself next time we iterate the bundled version of zlib et al?
 
 
#3 AnyCPU
09:48 May 01, 2014
In general I left this for history reasons because Microsoft sometime can be not predicable.  
 
#4 AlbrechtS
16:12 Sep 18, 2014
Hmm, scope is "3 - Applies to all machines and operating systems", but #3 mentions Microsoft explicitly. I thought (and I read lots of comments about this in the FLTK code) that this is a Microsoft-specific way to prepend '_' to the so-called (?) ISO functions.

See also this link:
http://lists.apple.com/archives/xcode-users/2011/Jan/msg00018.html

I would prefer not to use even more of these macros we discussed recently that are also mentioned in the article linked above, like

  #ifdef _WIN32
  #define stat _stat
  #endif

So, who does really know what this is all about? If it is only a warning in MS compilers, can this warning be suppressed?
 
 
#5 AlbrechtS
11:39 Jul 17, 2015
I believe this issue is solved since the upgrade of zlib from version 1.2.5 to 1.2.8 on 16 Mar 2015 (svn r 10624).

I can't test it, but I see:

$ grep -i -n -5 _close zlib/*.[chH]
zlib/gzguts.h-37-
zlib/gzguts.h-38-#ifdef WINAPI_FAMILY
zlib/gzguts.h-39-#  define open _open
zlib/gzguts.h-40-#  define read _read
zlib/gzguts.h-41-#  define write _write
zlib/gzguts.h:42:#  define close _close
zlib/gzguts.h-43-#endif
zlib/gzguts.h-44-
...

This should fix the issue for close and write.

]$ grep -i -n -5 lseek zlib/*.[chH]
zlib/gzlib.c-4- */
zlib/gzlib.c-5-
zlib/gzlib.c-6-#include "gzguts.h"
zlib/gzlib.c-7-
zlib/gzlib.c-8-#if defined(_WIN32) && !defined(__BORLANDC__)
zlib/gzlib.c:9:#  define LSEEK _lseeki64
zlib/gzlib.c-10-#else
zlib/gzlib.c-11-#if defined(_LARGEFILE64_SOURCE) && _LFS64_LARGEFILE-0
zlib/gzlib.c:12:#  define LSEEK lseek64
zlib/gzlib.c-13-#else
zlib/gzlib.c:14:#  define LSEEK lseek
zlib/gzlib.c-15-#endif
zlib/gzlib.c-16-#endif

This ought to fix the issue for lseek.

Can you confirm that the warnings are gone? If not, please specify exactly which version of compiler and toolchain, and show exact warnings.

Otherwise this STR will be closed in a few days.
 
 
#6 AlbrechtS
06:13 Jul 30, 2015
This STR has not been updated by the submitter for two or more weeks and has been closed as required by the FLTK Configuration Management Plan. If the issue still requires resolution, please re-submit a new STR.  
     

Return to Bugs & Features ]

 
 

Comments are owned by the poster. All other content is copyright 1998-2024 by Bill Spitzak and others. This project is hosted by The FLTK Team. Please report site problems to 'erco@seriss.com'.