TLV Redux

Alright @adamvoncorswant, feel free to take a peek when you get a moment. Never before has there been such a test-covered parser :tada: 

FYI @linnie if you're interested. EMV stuff and all that.
Ok, that my editor also flagged as inappropriate. Fixed that and another
unused variable. Stop commenting now.

On Mon, Dec 22, 2014 at 2:09 PM, Adam von Corswant <notifications@github.com
> wrote:
>
> In izettle-emv/src/test/java/com/izettle/tlv/TLVEncoderTest.java
> <https://github.com/iZettle/izettle-toolbox/pull/97#discussion-diff-22165120>
> :
>
> > +package com.izettle.tlv;
> > +
> > +import com.izettle.java.Hex;
> > +import org.junit.Assert;
> > +import org.junit.Test;
> > +
> > +/**
> > + * Created by fidde on 16/12/14.
> > + */
> > +public class TLVEncoderTest {
> > +
> > +	@Test
> > +	public void testLengthBounds() throws Exception {
> > +
> > +		TLVEncoder encoder = new TLVEncoder();
> > +		Assert.assertArrayEquals(encoder.encodeLength(0), Hex.hexToByteArray("00"));
>
> Ok. This is last comment. Promise :)
>
> The method encodeLength is static and need not an object to be called
> upon. So:
> TLVEncoder.encodeLength(0) would suffice. Then the created object the
> line above could be removed.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/iZettle/izettle-toolbox/pull/97/files#r22165120>.
>