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

Re: [PATCH 8/10] Fix some minor warnings to allow -Werror.

From
RRRocco Rutte <pdmef@gmx.net>
Date
Aug 2, 2006, 08:53 UTC
Message-ID
<20060802085330.GA6595@peter.daprodeges.fqdn.th-h.de>
In-Reply-To
<7vvepbvauo.fsf@assigned-by-dhcp.cox.net>
Hi,
* Junio C Hamano [06-08-02 00:46:23 -0700] wrote:
[...]
Show 10 quoted lines
>This kills the compilation with:
>
>gcc -o builtin-mailinfo.o -c -O2 -Werror -ansi -pedantic -std=c99 -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -Wall -Wdeclaration-after-statement -g -DSHA1_HEADER='<openssl/sha.h>' -DNO_STRLCPY builtin-mailinfo.c
>cc1: warnings being treated as errors
>builtin-mailinfo.c: In function 'convert_to_utf8':
>builtin-mailinfo.c:561: warning: passing argument 2 of 'iconv' from incompatible pointer type
>
>where the line 561 reads:
>
>	nrc = iconv(conv, &in, &insize, &out, &outsize);

It depends on the iconv() implementation as what the input pointer is defined: some use just 'char*' while others use 'const char*' (GNU libiconv IIRC).

   bye, Rocco
-- 
:wq!
Previous: Junio C HamanoNext: Ramsay Jones
Message 3 of 4 in “Fix some minor warnings to allow -Werror.”
  1. 8/10 Fix some minor warnings to allow -Werror.Ramsay Jones, Aug 2, 2006
  2. Junio C HamanoAug 2, 2006
  3. Rocco RutteAug 2, 2006
  4. Ramsay JonesAug 2, 2006

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.