git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH/RFC] Fix some warnings (on cygwin) to allow -Werror

From
Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Date
Jul 5, 2008, 20:56 UTC
Message-ID
<486FE00C.7020207@ramsay1.demon.co.uk>
In-Reply-To
<E4D3A379-9EB2-49D1-AD42-CFFC453A2B08@zib.de>
Steffen Prohaska wrote:
Show 28 quoted lines
> On Jul 3, 2008, at 5:52 PM, Ramsay Jones wrote:
> 
>> Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
>> ---
>>
>> Despite the subject line, this should be useful on any system for
>> which uint32_t is defined to be unsigned long rather than
>> unsigned int. (and where the return type of htonl() is similarly
>> defined).
> 
> For MINGW, we fixed these warnings in the compat layer, see cd800eecc2:
> 
> diff --git a/compat/mingw.h b/compat/mingw.h
> index a87cc96..6bc049a 100644
> --- a/compat/mingw.h
> +++ b/compat/mingw.h
> @@ -186,6 +186,10 @@ pid_t mingw_spawnvpe(const char *cmd, const char  
> **argv, char **env);
>   void mingw_execvp(const char *cmd, char *const *argv);
>   #define execvp mingw_execvp
> 
> +static inline unsigned int git_ntohl(unsigned int x)
> +{ return (unsigned int)ntohl(x); }
> +#define ntohl git_ntohl
> +
>   sig_handler_t mingw_signal(int sig, sig_handler_t handler);
>   #define signal mingw_signal
> 

Yes, I had something similar at one point (but not in compat). However, not all of these warnings are solved by the above. For example, the changes to builtin-fast-export.c, builtin-pack-objects.c, and pack-check.c do not involve ntohl() at all. (some hunks in other files do not involve ntohl() either).

When the PRIu32 macro idea was suggested (I think it was Simon 'corecode' Schubert who first mentioned it), I thought it would look so ugly, no one would agree to such a patch... Having implemented it, well I guess it's not quite as bad as I feared ;-) YMMV!

ATB,
Ramsay Jones
Previous: Steffen Prohaska
Message 7 of 7 in “Fix some warnings (on cygwin) to allow -Werror”
  1. Fix some warnings (on cygwin) to allow -WerrorRamsay Jones, Jul 3, 2008
  2. Junio C HamanoJul 4, 2008
  3. Ramsay JonesJul 5, 2008
  4. Junio C HamanoJul 6, 2008
  5. Ramsay JonesJul 8, 2008
  6. Steffen ProhaskaJul 4, 2008
  7. Ramsay JonesJul 5, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.