From 4b3bd4b739bd7a89c5dd972fff726556b219aeb2 Mon Sep 17 00:00:00 2001 From: Linus Torvalds Date: Tue, 30 Aug 2011 14:36:34 -0700 Subject: [PATCH] Fix stupid mis-initialization of current sample .. nice compiler warning hidden by the crazy gcc pointer sign warnings that nobody wants to see (yes, we really do want to do 'strlen()' even on unsigned strings, don't complain, crazy bitch compiler). So this also makes our CFLAGS set -Wno-pointer-sign. Signed-off-by: Linus Torvalds --- Makefile | 5 ++++- parse.c | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index f77a9faa4..1a2163784 100644 --- a/Makefile +++ b/Makefile @@ -1,2 +1,5 @@ +CC=gcc +CFLAGS=-Wall -Wno-pointer-sign -g + parse: parse.c - gcc -Wall -g -o parse `xml2-config --cflags` parse.c `xml2-config --libs` + $(CC) $(CFLAGS) -o parse `xml2-config --cflags` parse.c `xml2-config --libs` diff --git a/parse.c b/parse.c index bbe1a68ca..43bf82d86 100644 --- a/parse.c +++ b/parse.c @@ -185,7 +185,7 @@ static void sample_start(void) if (!dive) return; } - sample = dive->samples + nr; + sample = dive->sample + nr; } static void sample_end(void)