3 Tests Failing on ARM Cortex-M4 CPU about qcbor HOT 9 CLOSED

Hey Darren,

I'll look at this, but it may take a few days. This certainly should be tracked down and fixed. Thanks for reporting.

This is 32-bit, right?

from qcbor.

Hi Laurence,

Thanks for getting back to me. Yes, this is a 32-bit processor with limited FLASH and RAM, 1MB and 256KB. I say a post by Jared Wolff about using it.

Any thoughts on how QCBOR compares to TinyCBOR?

from qcbor.

Next questions:
what compiler?
what is SIZE_MAX?

from qcbor.

The compiler is from part of Segger Embedded Studio https://www.segger.com/products/development-tools/embedded-studio/

SIZE_MAX is set to INT32_MAX or 2147483647L

from qcbor.

I just pushed a fix that should take care of these two:
NotWellFormedTests FAILED (returned 2079)
DecodeFailureTests FAILED (returned 9620)

Nothing wrong with the actual implementation, just an issue with the way the tests were constructed. They expected SIZE_MAX to be at least UINT32_MAX.

I don't know about this yet:
BstrWrapErrorTest FAILED (returned -3)`
The version I just pushed will report more detail on that failure that might help me figure it out. It doesn't seem to be related to SIZE_MAX. Would appreciate if you re run the tests and tell me the results.

from qcbor.

I haven't looked very hard a tinyCBOR and when I did it was a while a go. Here's a few things I've tried to do in QCBOR:

• Clear documented public interface. You don't have to look at the internal code to use QCBOR
• Handle CBOR tag correctly. A messy topic that was only recently clarified. See RFC 8949.
• Easy install without #ifdefs
• Provide common code for common things needed when decoding, particularly map decoding

from qcbor.

I've pushed another fix that may solve the problem with BstrWrapErrorTest FAILED (returned -3)

I think it's related to SIZE_MAX < UINT32_MAX.

from qcbor.

@laurencelundblade Thanks for adding the updates. I should have time to test this today.

from qcbor.

Fixes were made that should have addressed the issue, so closing after waiting a month for confirmation.

from qcbor.