Re: gcc-4.1.2 miscompiles setedit from CVS

[Salvador E. T. <sal...@in...>]

On 20/08/06 07:45, Witold Filipczyk wrote:

>gcc-4.1.2 miscompiles setedit from CVS.
>The first error is trivial to fix. The patch in the attachment.
>  
>

Ok, I'm applying it.

>The second error is related to packed attribute in setedit/runprog.cc:
>
>g++  -I../include -I../setedit/include -I../settvuti/include
>-I../sdg/include -I../infview/include -I../../tvision/include -I../extra
>-I../easydiag -I../librhuti -I../mp3 -I../calcu -I../holidays
>-DFOR_EDITOR -O2 -pipe  -c ../setedit/runprog.cc -o obj/runprog.o
>../setedit/runprog.cc: In function 'void ConfigureRunCommand()':
>../setedit/runprog.cc:95: warning: 'packed' attribute ignored for field
>of type 'char [256]'
>../setedit/runprog.cc:99: warning: 'packed' attribute ignored for field
>of type 'char [4]'
>../setedit/runprog.cc:116: error: cannot bind packed field
>'box.ConfigureRunCommand()::<anonymous
>struct>::tl.TListBoxRec::selection' to 'ccIndex&'
>  
>

I don't understand it. Sure that char[4] is already 32 bits aligned but 
I don't want to assume that it means that this is also "aligned" for 64 
and 128 bits machines. So that's a silly warning. Is the error related 
to the warning? what happends if you remove the packed attribute to the 
above mentioned members?

SET

P.S. Be patient with my next reply! more in another mail