De_Interleaver was not properly calculating
when to stop, and the decode_bytes was working with
old assumptions on de_interleaver.

fixed both

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Fixes #24



Sometimes it makes sense to have incomplete data.
This way we both de-interleave the incomplete data
and we give a std::vector<bool> of which bytes were
transmitted correctly and which were zeroed out.

Accessible through the end_of_input() calls, as it is the
only call that makes sense for this, since we need to
also ignore any other repair symbol.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

The de_interleaver was still following the
raw esi (max_esi == extended_esi) instead of the
real number of symbols in the block.

Also a minor bugfix that probably does not change
anything in precode_solver

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

set default/delete for all class constructors

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Spaces, not tabs.

Actually, I liked tabs much more. But it seems a lot
of IDEs and general reader do not handle them correctly,
or do not let you configure "1 tab == 4 spaces", so the code
was not always readable.

"git diff -w" for this commit is en empty list.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Also: modify a bit the API for 32 bits.
The user should not expect RQ to manage more memory
than the 32 bit machine can handle. This means that
some methods should work with and return size_t, not uint64_t

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Modified the RAW API (again) to make it more usable.
"bytes" parameter ended up being confusing.

Cli tool should nw be able to ancode/decode properly

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Note: somehow there is still some interleaving.
Which is really bad. There's probably something
wrong in both the interleaver and de-interleaver.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Groundwork started.
Started moving things to RFC6330.
Next steps will be cleanup & RaptorQ namespaces.

Meanwhile we need better file naming.

Supporting multiple versions of the same API
should be easier now.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Also, fix corner cases for misaligned sizes

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

We might want to support more than one API version in the future.
Prepare for that.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

There seems to be support for it everywhere.
defines are bulky and error-prone

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

I almost made this a header-only library. should I make the switch now?

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

test_c did not correctly calculate the memory size, although it
was still fine due to the data types.

the De_Interleaver was reading not-initialized data in the
"skip > 0" case.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Multiblock alignment decoding was wrong if
the block size was not aligned with the output alignment.

Also, what we were calling output iterator was really a
forward iterator. Get it right.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

I'll try to remember to check old codepaths when moving
to new code :p

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

We were neither respecting sub-block differences,
neither counting the alignment, which is not
necessary the input data alignment...

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

The C++ test now can be easily moved to using different
alignments for testing or playing around.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Previous fix was hurried, and only fix one of the
possible cases. Found the real cause and fixed it.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Worst case: 1-byte alignment, decoding of
881GB of data. We need 40 bits.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Should never happen, but someone might request
a symbol of 2 bytes, with a 4byte alignment.
We check for that, but we might want to
be more flexible in the future

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

These errors... Just... Just disregard this commit, please?
I really need some test. Now please give me some time to code
proper tests and repent for these sins.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Make clang happy, let's compile with
-Weverything!

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>

Switched to iterators instead of std::vector.

Now it's possible to build the C interface, and
you can use any container you want.

Signed-off-by: Luca Fulchir <luker@fenrirproject.org>