Re: [Chromium-users] Re: [Chromium-dev] Chromium 1.9 second release candidate

[Eric S. <er...@sa...>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wed, 24 May 2006, Brian Paul wrote:
> Eric Sandall wrote:
>> On Wed, 24 May 2006, Brian Paul wrote:
>>> Brian Paul wrote:
>>>> A second release candidate of Chromium 1.9 is ready at
>>>> http://chromium.sf.net/beta
>>>>=20
>>>> This just accumulates a few more bugs fixes since the first RC.
>>>=20
>>> I haven't seen any feedback on this, positive or negative.
>>>=20
>>> It would be good to at least have a couple people confirm that they're=
=20
>>> tried
>>> it and it's OK.
>>>=20
>>> Thanks.
>>=20
>> Trying to compile cr-1.9rc2 gives:
>> $ make
>> ...
>> Compiling state_program.c
>> cc1: warnings being treated as errors
>> state_program.c: In function =91crStateGetVertexAttribivNV=92:
>> state_program.c:1042: warning: =91floatParams[0]=92 is used
>> uninitialized in this function
>> state_program.c:1040: warning: =91floatParams[1]=92 may be used
>> uninitialized in this function
>> state_program.c:1040: warning: =91floatParams[2]=92 may be used
>> uninitialized in this function
>> state_program.c:1040: warning: =91floatParams[3]=92 may be used
>> uninitialized in this function
>> gmake[2]: *** [../built/crstate/Linux/state_program.o] Error 1
>> gmake[1]: *** [dep] Error 2
>> make: *** [state_tracker.subdir] Error 2
>>=20
>> This is with gcc 4.1.0-3 on Fedora Core 5. If I remove -Werror from
>> configs/Linux.mk CXXFLAGS and CFLAGS cr-1.9rc2 compiles (I have not
>> tried running it yet).
>
> OK, I actually fixed the same problem in a different function and didn't=
=20
> realize there was a second instance of it.
>
> The patch is attached.

Still gives:
Rebuilding dependencies for state_program.c
Compiling state_program.c
cc1: warnings being treated as errors
state_program.c: In function =E2=80=98crStateGetVertexAttribivARB=E2=80=99:
state_program.c:1099: warning: =E2=80=98floatParams[0]=E2=80=99 is used
uninitialized in this function
state_program.c:1097: warning: =E2=80=98floatParams[1]=E2=80=99 may be used
uninitialized in this function
state_program.c:1097: warning: =E2=80=98floatParams[2]=E2=80=99 may be used
uninitialized in this function
state_program.c:1097: warning: =E2=80=98floatParams[3]=E2=80=99 may be used
uninitialized in this function
gmake[2]: *** [../built/crstate/Linux/state_program.o] Error 1
gmake[1]: *** [dep] Error 2
make: *** [state_tracker.subdir] Error 2

And again when the above is fixed:
Rebuilding dependencies for state_program.c
Compiling state_program.c
cc1: warnings being treated as errors
state_program.c: In function =E2=80=98crStateGetVertexAttribdvARB=E2=80=99:
state_program.c:1113: warning: =E2=80=98floatParams[0]=E2=80=99 is used
uninitialized in this function
state_program.c:1111: warning: =E2=80=98floatParams[1]=E2=80=99 may be used
uninitialized in this function
state_program.c:1111: warning: =E2=80=98floatParams[2]=E2=80=99 may be used
uninitialized in this function
state_program.c:1111: warning: =E2=80=98floatParams[3]=E2=80=99 may be used
uninitialized in this function
gmake[2]: *** [../built/crstate/Linux/state_program.o] Error 1
gmake[1]: *** [dep] Error 2
make: *** [state_tracker.subdir] Error 2

Seems to be fixed in CVS. Attached is a patch (after applying yours)
that fixes the other two locations.

- -sandalle

- --
Eric Sandall                     |  Source Mage GNU/Linux Developer
er...@sa...                  |  http://www.sourcemage.org/
http://eric.sandall.us/          |  SysAdmin @ Inst. Shock Physics @ WSU
http://counter.li.org/  #196285  |  http://www.shock.wsu.edu/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2.2 (GNU/Linux)

iD8DBQFEdL/THXt9dKjv3WERAgeAAJ9RfL1vAsDNGxiJ/1C3HJ9RVIdgeQCfbCNG
GAsQAuWsKhyTqe7KW3io7kY=3D
=3Dwh1q
-----END PGP SIGNATURE-----