[PATCH v2 2/4] widl: Support WinRT parameterized interface type.

Jacek Caban jacek at codeweavers.com
Wed Feb 17 06:40:36 CST 2021


Hi Rémi,

On 16.02.2021 21:15, Rémi Bernon wrote:
> +    case STMT_IMPORT:
> +    case STMT_IMPORTLIB:
> +    case STMT_PRAGMA:
> +    case STMT_CPPQUOTE:
> +        fprintf(stderr, "%d\n", stmt->type);
> +        assert(0);
> +        break;


In general, we shouldn't assert or crash on invalid or unsupported 
syntax. In this particular case, I don't think there is anything special 
we need to do about those statements, so it seems to me that removing 
fprintf() and assert() is all we need for those statements. However, I 
was also able to crash widl by adding a struct declared inside templated 
interface (which should likely be not supported anyway).


Thanks,

Jacek




More information about the wine-devel mailing list