03:57 <anthonybullard> I think I've finally uncovered the rough spots in the json API.

03:58 <SeanTAllen> all of it anthonybullard ?

03:58 <anthonybullard> No just one :-)

03:58 <anthonybullard> one part that is

03:58 <SeanTAllen> we have different opinions and that is ok

03:59 <anthonybullard> I guess I wonder why JsonType's member types all have `val` but JsonObject and JsonArray are `ref`

04:43 <anthonybullard> I was able to resolve that issue, but running into a not incredibly helpful backtrace from the compiler, even in debug mode

04:48 <anthonybullard> https://gist.github.com/gamebox/a6048c187538044a5a8d210e1c623619

09:01 <profetes> vaninwagen: thank you very much, looks Promise'ing, I'll try to implement this into my solution later. thanks!

09:03 <vaninwagen> anthonybullard: could you create a github issue with your code and a backtrace from a debug build of ponyc? Which version did you use.

09:03 <vaninwagen> This is clearly a compiler bug

09:10 <vaninwagen> profetes: nice pun, hard to avoid actually

15:29 <Candle> https://github.com/CandleCandle/text-reader I think I've covered all the edge cases (see the tests, which all pass on my machine and 0.25.0). Apart from the abuse of @printf, can anyone see anywhere that could be improved? I'm about to try to use it more in anger so I'm sure that I'll find some...

15:31 <Candle> It's intended as a replacement for buffered/Reader.line() where we know that (a) the data comes in in chunks and (b) lines are always separated by \r\n.

15:41 <vaninwagen> Candle nice! When using it to parse http headers, how could i go about getting the whole body?

15:42 <vaninwagen> The only thing that jumps into my face is the copying of the line string

16:04 <Candle> If I understand your question, that is what the 'remaining()' function is there for; grab the unused buffer elements to read the body.

16:05 <Candle> I think we came to the conclusion that without re-implementing the String class to work with multiple underlying arrays, we have to copy the bytes into a new String.

16:09 <Candle> Part of the problem is that one call of the data array passed received(...) can contain 0..n lines. (see the test named 'line-reader/complete-example')

16:10 <Candle> I take it you've seen the psudo-code near the start of text-reader.pony ?

16:17 <vaninwagen> Yep

16:19 <vaninwagen> I think if we use something like a rope, that is a pointer with a length into a bunch of arrays and make it Stringable and Comparable to String and other such pointers, we have a good foundation for reducing copying altogether

16:19 <vaninwagen> I was just thinking a lot about this lately

16:20 <vaninwagen> Your text-reader is a great step forward

18:15 <travis-ci> ponylang/ponyc#5515 (master - 8a0a7d4 : Stephan Renatus): The build was fixed.

18:15 <travis-ci> Build details : https://travis-ci.org/ponylang/ponyc/builds/467107320

18:15 <travis-ci> Change view : https://github.com/ponylang/ponyc/compare/ae7d070a0d2c...8a0a7d46bffb

19:14 <travis-ci> ponylang/ponyc#5516 (master - 8a0a7d4 : Stephan Renatus): The build was broken.

19:14 <travis-ci> Change view : https://github.com/ponylang/ponyc/compare/ae7d070a0d2cb3fc24c9c7960de108be1ff26d1d...8a0a7d46bffbfd718dee473f47eba01078eecf8a

19:14 <travis-ci> Build details : https://travis-ci.org/ponylang/ponyc/builds/467131135

20:09 <aturley> streaming advent of code in pony here if anyone wants to watch ... https://www.twitch.tv/aturls/