search

sammielove45 / propgcc

Automatically exported from code.google.com/p/propgcc
1 stars 1 forks source link

Probable typo in lib/drivers/dosfs.c wastes a small amount of space #29

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The DFS_GetPtnStart() routine at the top of dosfs.c is conditionally included 
via "#ifndef PROPGCC_MODE" conditional.  However, the rest of dosfs.c and 
dosfs.h use "#ifndef PROPGCC_MODS" to trim off code that PropGCC does not need.

(Note that the comment at the start of dosfs.c also mentions PROPGCC_MODE, 
while the comment at the start of dosfs.h mentions PROPGCC_MODS.)

I checked the rest of the code and nothing seems to call DFS_GetPtnStart.

This problem really quite trivial, but fixing it will free up 344 bytes to 
hapless hackers like myself trying to use a SD card in LMM mode. 

What version of the product are you using? On what operating system?

propgcc-0.2.2

Original issue reported on code.google.com by tjstefan...@charter.net on 5 Feb 2012 at 8:28

GoogleCodeExporter commented 9 years ago 
I fixed this while adding SDHC functionality so I could get the extra code 
space.

Original comment by tjstefan...@charter.net on 3 Apr 2012 at 9:08