Dan Hipschman wrote:
+ write_type_decl_left(fp, mdef->type); + fprintf(fp, " CALLBACK %s_", iface->name); + write_name(fp, mdef); + fprintf(fp, "_Proxy(\n"); + write_args(fp, m->args, iface->name, 1, TRUE); + fprintf(fp, ")"); + if (body) { + type_t *rt = mdef->type; + fprintf(fp, "\n{\n"); + fprintf(fp, " %s\n", comment); + if (rt->name && strcmp(rt->name, "HRESULT") == 0) + fprintf(fp, " return E_FAIL;\n");
It would be better to make the default implementation of the stubs return E_NOTIMPL instead of E_FAIL, since the former is a lot more specific and makes identifying the source of misbehaviour of an application easier. -- Rob Shearman