patchAVR Downloader/UploaDEr - Patches: patch #8436, Remove HAVE_STDINT_H guards in...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #8436: Remove HAVE_STDINT_H guards in flip1.c and flip2.c

Submitter:  Rok Mandeljc <mrok>
Submitted:  Thu 17 Apr 2014 11:40:48 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Wont Do Privacy:  Public
Assigned to:  None Open/Closed:  Closed

Thu 18 Jan 2018 10:16:36 PM UTC, comment #2: 

"Need info" timed out after more than three years.

Joerg Wunsch <joerg_wunsch>
Group administrator
Tue 25 Nov 2014 08:16:44 PM UTC, comment #1: 

I cannot confirm your statement #1: it compiles fine the way
it is in my MinGW32 cross-compilation environment.

Regarding reasoning #2, I don't see we are consistent in
either direction:

. libavrdude.h checks for HAVE_STDINT_H, and acts correctly
  upon it
. fileio.c and ft245.c use <stdint.h> unchecked
. flip1/2 checks for either symbol, and acts upon it

IMHO, it would make most sense to remove all the checks
(including the one in libavrdude.h), and then assume <stdint.h>
always being present.  AVRDUDE assumes a C99 compatible
environment in many other places already anyway.

Would you be willing to resubmit a patch that unifies it that
way?

Joerg Wunsch <joerg_wunsch>
Group administrator
Thu 17 Apr 2014 11:40:48 AM UTC, original submission:  

The attached patch removes HAVE_STDINT_H (and HAVE_INTTYPES_H) guards from flip1.c and flip2.c. The reason is two-fold:

a) HAVE_STDINT_H does not appear to be defined under MinGW*; consequently, stdint.h is not included, and compilation fails due to uint32_t being undefined

b) the rest of source files that include stdint.h do not use the guards, either


[*] using win-builds bundle 1.3.0 and msys (from MinGW-builds)

Rok Mandeljc <mrok>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #31187:  stdint-fix.patch added by mrok (1KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by joerg_wunsch (Posted a comment)
  • -email is unavailable- added by mrok (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2018-01-18 joerg_wunsch StatusNeed Info Wont Do
        Open/ClosedOpen Closed
    2014-11-25 joerg_wunsch StatusNone Need Info
    2014-04-17 mrok Attached File- Added stdint-fix.patch, #31187

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code