>>>>> "Tony" == Tony Luck <Luck> writes:

>> - if (ia64_pal_mem_attrib(&attrib) != 0) - return 0; + if
>> (ia64_pal_mem_attrib(&attrib) != 0){ + *p++ = '\n'; + return
>> p-page; + }

Tony> Could we be even more tolerant with a series (gasp) goto
Tony> statements instead of the "return"?  Like this:

Tony>   if (ia64_pal_mem_attrib(&attrib) != 0) goto skip_mem_attrib;

Tony>   ... code to print things in &attrib

Tony> skip_mem_attrib:


Tony> This would require a bit more re-arranging of the code to make
Tony> all the uses of each PAL call fall right after the call.


Can do... Give me a while to whip up and test a patch...

PeterC 
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to