freemyipod r23 - Code Review

Jump to: navigation, search
Repository:freemyipod
Revision:r22‎ | r23 | r24 >
Date:04:20, 5 August 2010
Author:farthen
Status:new
Tags:
Comment:
Merge r18 and r19 into branches/4g_compat.
Modified paths:
  • /embios/branches/4g_compat (modified) (history)
  • /embios/branches/4g_compat/Makefile (modified) (history)
  • /embios/branches/4g_compat/strlen.c (modified) (history)

Diff [purge]

Index: embios/branches/4g_compat/strlen.c
@@ -22,15 +22,13 @@
2323
2424
2525 #include "global.h"
26 -#include <string.h>
27 -#include <limits.h>
2826
29 -size_t strlen(_CONST char *str)
 27+
 28+size_t strlen(const char *str)
3029 {
31 - _CONST char *start = str;
 30+ const char *start = str;
3231
33 - while (*str)
34 - str++;
 32+ while (*str) str++;
3533
3634 return str - start;
3735 }
Index: embios/branches/4g_compat/Makefile
@@ -8,7 +8,7 @@
99 OBJCOPY := $(CROSS)objcopy
1010 UCLPACK := ucl2e10singleblk
1111
12 -CFLAGS ?= -Os -fno-pie -fno-stack-protector -fomit-frame-pointer -I. -ffunction-sections -fdata-sections -std=gnu99
 12+CFLAGS ?= -Os -fno-pie -fno-stack-protector -fomit-frame-pointer -I. -ffunction-sections -fdata-sections
1313 LDFLAGS ?= "$(shell $(CC) -print-libgcc-file-name)" --gc-sections
1414
1515 preprocess = $(shell $(CC) $(PPCFLAGS) $(2) -E -P -x c $(1) | grep -v "^\#" | sed -e "s:^..*:$(dir $(1))&:")
Index: embios/branches/4g_compat
Property changes on: embios/branches/4g_compat
___________________________________________________________________
Added: svn:mergeinfo
## -0,0 +0,1 ##
1616 Merged /embios/trunk:r18-19

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r18Revert r12 and r13. They are unneccessary and might have nasty side effects.theseven00:04, 5 August 2010
r19Simplify strlen.ctheseven00:06, 5 August 2010