Rockbox mail archive

• Next message: Chris Holt: "Re: BATTERY-FAQ and the FM"
• Previous message: Martin: "Re: BATTERY-FAQ and the FM"
• In reply to: TP Diffenbach: "Patch #706111 id3 tags, implemented! But an error appears!"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] [ attachment ]

TP Diffenbach wrote:
> But changing 'sizeof( "TCON" ) - 1' to that expression's value, '4', is I
> think a loss. '4' by itself doesn't document anything, and certainly doesn't
> indicate to the reader what the number is used for. By showing the
> expression from which 4 results, the user knows that it relies on the length
> of the string "TCON" less the nul terminator, and is not something else
> entirely.

The name of the field is 'tag_length', which I think fully communicates the purpose of it. I removed the sizeof() construct simply because I found it made the code unneccessarily messy.

> While not making it impossible to make errors, this I think makes it easier
> for coders adding tags, by relegating tediously repeated code to the macro.

I dislike macros that hide code. It is in my opinion mostly better to have slightly more complex code if it avoids having to pre-process it in your head.

I also do not like to create language constructs to protect the programmer from himself. Errors will always happen, but they are much simpler to find and fix if the code just does the job straight with as little fuss as possible.

-- 
Björn

• Next message: Chris Holt: "Re: BATTERY-FAQ and the FM"
• Previous message: Martin: "Re: BATTERY-FAQ and the FM"
• In reply to: TP Diffenbach: "Patch #706111 id3 tags, implemented! But an error appears!"
• Messages sorted by: [ date ] [ thread ] [ subject ] [ author ] [ attachment ]