Dear Gerlando Falauto,
In message 4EE5CA38.6090807@keymile.com you wrote:
if (process_var(name, nvars, vars)&& hdelete_r(name, htab) == 0) debug("DELETE ERROR ##############################\n");
This is incorrect indentation.
I think it's easier to read it the original way, and it should not make any difference as far as code size is concerned.
The Coding Style makes an explicit exception regarding the line length for user visible strings:
83 Statements longer than 80 columns will be broken into sensible chunks, unless 84 exceeding 80 columns significantly increases readability and does not hide 85 information. Descendants are always substantially shorter than the parent and 86 are placed substantially to the right. The same applies to function headers 87 with a long argument list. However, never break user-visible strings such as 88 printk messages, because that breaks the ability to grep for them.
himport_r() is getting a bit overloaded,
Actually, I believe it makes no longer sense to have it called "_r", as it was the original reference to the function being recursively calleable (i.e. reentrant) as opposed to other versions which were not.
Has this changed?
Also, for me this patch adds 500 bytes. I wonder if more of the code could made optional?
Frankly, I'm surprised to hear this adds that much overhead. Actually, I can't see this increase in code size as you mention. What architecture are you referring to? In my workspace (ppc_6xx) u-boot.bin and a stripped u-boot ELF file are surprisingly unchanged in size, even with debug #defined!
Don't look at file size. Use the "size" command and compare code / data / bss sizes.
Best regards,
Wolfgang Denk