On Fri Mar 24 20:14:07 2023 +0000, Connor McAdams wrote:
The idea of something like `bstrcat_realloc()` is probably a better idea, I hadn't considered that. I'm not sure I follow on the issue of BSTR's not being NULL terminated, the values I'm passing in are all based on string length without the NULL terminator. `SysReAllocStringLen` allocates enough space for the passed in length and a NULL terminator.
Well, unless the confusion is around the size doubling behavior. That was mainly done to match other dynamic array functions in wine, where AFAIU it's more performant to double the size of an allocation instead of doing realloc every time. It might be unnecessary here.