Josh Triplett wrote: > Ramsay Jones wrote: >> When updating this patch from the 0.3 version, I made the (new) third parameter >> KW_ATTRIBUTE only, rather than (KW_ATTRIBUTE | KW_ASM), since it did not seem >> correct to allow an asm there; is that correct? > > Correct; I don't think you can have an asm there. > OK, good. >> The test case for this was abstracted from an example in the "expat.h" header file. >> >> $cat ape.c >> >> typedef void (__attribute__((__cdecl__)) *FP)(void *u, const char *n); >> >> void set_FP(void *cb, FP f); > > Applied. I think you just made a Wine developer very happy. Michael, this > patch fixes the test case you provided for Wine; I applied your patch for the > new test case. > > Thanks, Ramsay! > My pleasure. ATB, Ramsay Jones - To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
