Increase key length limit to 120.

[acherkashyn]

Increase the key length limit and update unit tests accordingly.

[JakeWharton]

I'm wondering if enforcing a length is useful at all. @swankjesse?

[swankjesse]

If we were ever to switch to a binary encoding, we'd want a length limit like 255 so we can store the length in a single byte. And because the key name is used as a filename we shouldn't let it go arbitrarily long.

But 120 should be fine.

[gubatron]

really cool observation @swankjesse (in that case just raise it now to 255 then)

[JakeWharton]

The file name limit is 127 on Android. 120 is good.
On Jul 22, 2014 6:49 PM, "Angel Leon" notifications@github.com wrote:

really cool observation @swankjesse https://github.com/swankjesse (in
that case just raise it now to 255 then)

—
Reply to this email directly or view it on GitHub
#65 (comment)
.

[acherkashyn]

So then... merge? :)

[JakeWharton]

Squashed and merged.