[Nouveau] [PATCH 1/2] nouveau/compiler: fix trivial compiler warnings
Tobias Klausmann
tobias.johannes.klausmann at mni.thm.de
Wed Jul 8 12:34:25 PDT 2015
Mh i'm not aware of me ever changed the "nouveau_compiler". But i'm
happy to see this made you laugh, so it has something positive at
least... :/
On 08.07.2015 21:31, Emil Velikov wrote:
> Curious how many times this needs to come along, for you to change
> your mind. :-)
>
> On 8 July 2015 at 19:38, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
>> Compiler is wrong.
>>
>> On Wed, Jul 8, 2015 at 2:27 PM, Tobias Klausmann
>> <tobias.johannes.klausmann at mni.thm.de> wrote:
>>> nouveau_compiler.c: In function ‘main’:
>>> nouveau_compiler.c:216:27: warning: ‘code’ may be used uninitialized in
>>> this function [-Wmaybe-uninitialized]
>>> printf("%08x ", code[i / 4]);
>>> ^
>>> nouveau_compiler.c:215:4: warning: ‘size’ may be used uninitialized in
>>> this function [-Wmaybe-uninitialized]
>>> for (i = 0; i < size; i += 4) {
>>>
>>> Signed-off-by: Tobias Klausmann <tobias.johannes.klausmann at mni.thm.de>
>>> ---
>>> src/gallium/drivers/nouveau/nouveau_compiler.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/src/gallium/drivers/nouveau/nouveau_compiler.c b/src/gallium/drivers/nouveau/nouveau_compiler.c
>>> index 8660498..ca128b5 100644
>>> --- a/src/gallium/drivers/nouveau/nouveau_compiler.c
>>> +++ b/src/gallium/drivers/nouveau/nouveau_compiler.c
>>> @@ -144,7 +144,7 @@ main(int argc, char *argv[])
>>> const char *filename = NULL;
>>> FILE *f;
>>> char text[65536] = {0};
>>> - unsigned size, *code;
>>> + unsigned size = 0, *code = NULL;
>>>
>>> for (i = 1; i < argc; i++) {
>>> if (!strcmp(argv[i], "-a"))
>>> --
>>> 2.4.5
>>>
>>> _______________________________________________
>>> Nouveau mailing list
>>> Nouveau at lists.freedesktop.org
>>> http://lists.freedesktop.org/mailman/listinfo/nouveau
>> _______________________________________________
>> Nouveau mailing list
>> Nouveau at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/nouveau
More information about the Nouveau
mailing list