Jakob Borg

Besides, this is really only a thing here due to how the benchmarks are written. A caller can get the same behavior by doing `dst, err = Encode(dst[:cap(dst)], testfile)` anyway,...

Would you accept a PR to add a function `lz4.DecodeLength(dst, src []byte, length int) ([]byte, error)`? This function would operate on data that doesn't have the four byte preamble. The...

Yes, would be helpful. (It's going to be somewhat tricky at best, since we don't really know how much we need to fetch until a bit later. That a 1...

It depends. When merging GitHub will suggest a concatenation of all the commit messages in the branch. If that looks like shit I generally remove the crap. If that leaves...

That's awesome! I'll look into this and answer questions. In the meantime, after just glancing at your description, I think it would be nice to axe the http basic auth...

This is a great addition, long overdue. However, I think we should do it a bit more manually, and the `--paths` output is maybe not a great model. Looking at...

Okay fair enough, in that case how about grouping by directory? ``` In /home/acolomb/src/syncthing/devtest-home: Cert File: cert.pem Config: config.xml ... In /home/acolomb/src: Gui Assets: gui ... ``` I'm trying to...

The support bundle thing is specific functionality that is only available when debugging is enabled, yeah, not really related. Frankly I think "About" is a good place. (I also think...

In the spirit of the above bikeshedding, why are we bolding the database directory (and log file if there isn't one)? We should absolutely mention where the database lives, but...

I think that having the event data structs declared by the package triggering the event might be *conceptually* cleanest, but will almost certainly result in dependency cycle issues. The easiest...