Ian Denhardt

Results 430 comments of Ian Denhardt

What if we gave the request object a `Release()` method, which: 1. Releases the underlying message, if the request has not already been sent (if it has, `Send()` will have...

I opened a separate issue for the `errWriter` inspired API; I think these are orthogonal concerns.

Looks like this is fixed as of 1.0.2, but the version of `github.com/json-iterator/go`you're using has a lower version bound on this package; if you update the latter package to v1.1.12...

It looks like those attributes are for controlling whether directories can be submitted as input; not setting them does not prevent directories from being submitted. I came across this looking...

I think I'm unlikely to find time to work on it myself at least in the short to medium term. And I'm probably the wrong person for the job --...

Right now (1) is working pretty well for me, and I don't really want to add another source of mystery binaries to my system, or have to build a large...

Quoting Kenton Varda (2020-09-12 15:28:57) > FWIW, Cap'n Proto already has a newer, better ByteStream: > https://github.com/capnproto/capnproto/blob/master/c++/src/capnp/compat/byte-stream.capnp It's kindof sad that we have two different interfaces for this, when it...

> Arguably it could make sense to bake ping() more deeply into the capnp protocol, as I've seen lots of interfaces that end up with a method like this. I...

I suspect the node version is the issue; I see the same error on node 15 on Archlinux. It's not surprising to me it doesn't work on node 15 yet.

Note that there is a node14 branch that should work with Node 14 (but not 15).