Ji-Xinyou commented on PR #3479:
URL:
https://github.com/apache/incubator-opendal/pull/3479#issuecomment-1793283516
> > I think this break the current API, so my plan would be changing this
after our first release.
>
> Can you share more details about this decision? Why not change now instead?
For example, now our error is backed by `opendal_byte`. Users typically use
this by
```C
opendal_byte *b = error->message;
for (int i = 0; i < b->len; ++I)
printf("%s", b->data[i]);
opendal_byte_free(b)
```
But in the CString way, we are only giving a `char*`, user just do
```C
char* msg = error->message:
printf(%s", msg);
free(msg);
```
This will break the current way of handling error, but it is more
conventional
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
