{"thread":{"id":"7125","subject":"[PATCH 0/9] misc. cleanups","startedAt":"2007-03-07T01:44:04Z","lastAt":"2007-03-07T01:44:04Z","messageCount":1,"participants":["Shawn O. Pearce"],"isPatch":true,"patchVersion":1,"patchTotal":9},"messages":[{"id":"36515","messageId":"20070307014403.GA26482@spearce.org","threadId":"7125","inReplyTo":null,"subject":"[PATCH 0/9] misc. cleanups","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2007-03-07T01:44:04Z","receivedAt":"2007-03-07T01:44:04Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"This is a series of various cleanups that I have pending at the\nfront of my pack v4 topic.  The earlier ones are annoying nits,\nthe later ones fix compiler errors caused by compiling with:\n\n--snip--\n  CFLAGS += -Wall -ansi -pedantic -std=c99\n  \n  ifdef strict\n          CFLAGS += -Werror\n          CFLAGS += -Wfour-char-constants\n          CFLAGS += -fnon-lvalue-assign\n          CFLAGS += -Wdeclaration-after-statement\n          CFLAGS += -Wpointer-arith\n          CFLAGS += -Wbad-function-cast\n          CFLAGS += -Wcast-align\n          CFLAGS += -Wwrite-strings\n          CFLAGS += -Wshorten-64-to-32\n  \n          # No, currently a huge set.\n          #CFLAGS += -Wsign-compare\n  \n          # No, only in diffcore-rename and that's ok.\n          #CFLAGS += -Wfloat-equal\n  \n          # No, unused parameters, -Wsign-compare, missing initializers.\n          # CFLAGS += -Wextra\n  \n          # No, memcmp trips on signed/unsigned conversion.\n          # CFLAGS += -Wconversion\n  endif\n--snap--\n\n-- \nShawn.\n"}]}