Re: [PR] PROTON-2785: Fix pn_data_clear not clearing error [qpid-proton]

2024-07-30 Thread via GitHub
astitcher merged PR #431: URL: https://github.com/apache/qpid-proton/pull/431 -- 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:

Re: [PR] PROTON-2785: Fix pn_data_clear not clearing error [qpid-proton]

2024-07-30 Thread via GitHub
astitcher commented on code in PR #431: URL: https://github.com/apache/qpid-proton/pull/431#discussion_r1697201250 ## c/src/core/codec.c: ## @@ -431,6 +431,7 @@ void pn_data_clear(pn_data_t *data) data->current = 0; data->base_parent = 0; data->base_current = 0;

Re: [PR] PROTON-2785: Fix pn_data_clear not clearing error [qpid-proton]

2024-07-29 Thread via GitHub
PatrickTaibel commented on code in PR #431: URL: https://github.com/apache/qpid-proton/pull/431#discussion_r1696345652 ## c/src/core/codec.c: ## @@ -431,6 +431,7 @@ void pn_data_clear(pn_data_t *data) data->current = 0; data->base_parent = 0; data->base_current =

Re: [PR] PROTON-2785: Fix pn_data_clear not clearing error [qpid-proton]

2024-07-29 Thread via GitHub
astitcher commented on code in PR #431: URL: https://github.com/apache/qpid-proton/pull/431#discussion_r1695772565 ## c/src/core/codec.c: ## @@ -431,6 +431,7 @@ void pn_data_clear(pn_data_t *data) data->current = 0; data->base_parent = 0; data->base_current = 0;

[PR] PROTON-2785: Fix pn_data_clear not clearing error [qpid-proton]

2024-07-29 Thread via GitHub
PatrickTaibel opened a new pull request, #431: URL: https://github.com/apache/qpid-proton/pull/431 We've also hit the bug of the Go library that is described in PROTON-2785. This bug happens when one large message is sent and another message with arbitrary size gets marshaled.