On Mon, 17 Oct 2022, Michael Van Canneyt via fpc-pascal wrote:
On Mon, 17 Oct 2022, Hairy Pixels via fpc-pascal wrote:
For the JSON RPC library I wanted to supported dynamic arrays but in order
to do this I needed to duplicate TJSONStreamer's ObjectToJSON and
StreamProperty in order to bypass some exceptions. If StreamProperty was
virtual I could simply override that and stream the dynamic arrays before
calling inherited.
What do you think about that?
It would be a much better idea to implement the streaming of dynamic arrays
directly. Then the exceptions are maybe not necessary in the first place...
Small follow-up:
If you want to extend it, you can find the necessary code in the restbase
unit. It already implements the necessary code to (de)stream arrays, plus
some other enhancements as well. It's on my TODO list to merge the
implementations taking TJSONStreamer as a base, but if you are busy with
this maybe you can provide a patch...
Michael.
_______________________________________________
fpc-pascal maillist - fpc-pascal@lists.freepascal.org
https://lists.freepascal.org/cgi-bin/mailman/listinfo/fpc-pascal
