Remove semicolon at the end of #define

[deepikabhavnani]

Description

Remove semicolon followed by pre-processor defines. Noticed build failures when building code with "EVR_RTX_DISABLE" set.

Pull request type

[X] Fix
[ ] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

Note: Fix in CMSIS repo is present.

[cmonr]

Which code was erroring/failing before this fix?

[0xc0170]

Note: Fix in CMSIS repo is present.

Worth referring to the specific SHA/pull request. I can't locate it on develop branch there. Seems still wrong to me.

This should be fixed first upstream : https://github.com/ARM-software/CMSIS_5/blame/develop/CMSIS/RTOS2/RTX/Include/rtx_evr.h#L836 ?

@JonatanAntoni

[0xc0170]

I sent upstream fix as I found these extra ; there (compare to the rest of the file, should not be there)

[0xc0170]

Upstream fixed (It was merged)

[0xc0170]

As now upstream is fixed, this will be in when we update

[bulislaw]

In the future please reference the upstream commit in the commit description.

[0xc0170]

/morph build

[mbed-ci]

Build : SUCCESS

Build number : 2469
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7355/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

[mbed-ci]

Exporter Build : FAILURE

Build number : 2098
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7355/

[deepikabhavnani]

In the future please reference the upstream commit in the commit description.

I left the note in description "Note: Fix in CMSIS repo is present..." Not sure why grep on my local repo didn't show me semicolons in upstream (may be old copy / wrong command). Lesson learned will always check the web GitHub repo instead local repo.

@0xc0170 - Thanks for adding PR to CMSIS.

[deepikabhavnani]

Which code was erroring/failing before this fix?

EventRecorder code, when disabling RTX specific events

[cmonr]

Pipe closed.

/morph export-build

[mbed-ci]

Exporter Build : FAILURE

Build number : 2101
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7355/

[mbed-ci]

Test : SUCCESS

Build number : 2246
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/7355/2246

[cmonr]

/morph export-build

[mbed-ci]

Exporter Build : FAILURE

Build number : 2108
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7355/

[cmonr]

ಠ_ಠ

/morph export-build

[mbed-ci]

Exporter Build : SUCCESS

Build number : 2111
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7355/

[0xc0170]

I added the upstream fix to the merge commit msg