search

ARMmbed / core-util

DEPRECATED: Mbed 3 utilities library
Other
12 stars 17 forks source link

Allocator fix #17

Closed bogdanm closed 9 years ago

bogdanm commented 9 years ago

The implementation on top of atomic_ops was not completely correct. Also updated tests.

bogdanm commented 9 years ago

Should fix https://github.com/ARMmbed/mbed-sdk-private/issues/56 and probably other things.

bogdanm commented 9 years ago

@bremoran @rgrover @0xc0170

0xc0170 commented 9 years ago

LGTM

bremoran commented 9 years ago

Looks fine by inspection. Which platforms/toolchains/tests have you tested it with?

rgrover commented 9 years ago

does it fix the 'cpp' test that was hanging earlier?

bogdanm commented 9 years ago

Tested with frdm-k64f-gcc. Should fix the 'cpp' test, didn't try it locally. But the fix is needed regardless of that.

rgrover commented 9 years ago

published v0.0.11

bremoran commented 9 years ago

Confirmed; this fixes ARMmbed/mbed-sdk-private#56

0xc0170 commented 9 years ago

:dancers: