SZEDER Gábor <szeder.dev@xxxxxxxxx> writes: >> Now, the proper thing to do would be to switch to `size_t`. But this >> would make us deviate from the "upstream" code even further, > > This is not entirely true: upstream already uses 'size_t', so the > switch would actually bring our copy closer to upstream. Ah, earlier I said that within the context how kwset uses obstack, it is perfectly proper to fix it like the patch in question did, but the upstream already using size_t changes the picture quite a bit. I do not mind updating our copy of obstack, but make sure you pick the version with license compatible with ours. Thanks.
