>>>>> "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