Skip to content

Commit acee77b

Browse files
authored
FreeRTOS Kernel Spelling Update (#170)
* FreeRTOS Kernel Spelling Update * Added spell check to kernel repository. * Fixed small spelling errors in various kernel source files. * Added documentation for spellcheck. Note: Only kernel files are checked for spelling, and portable files are ignored.
1 parent 2428081 commit acee77b

File tree

7 files changed

+3130
-7
lines changed

7 files changed

+3130
-7
lines changed

.github/workflows/ci.yml

Lines changed: 37 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,37 @@
1+
name: CI Checks
2+
on:
3+
push:
4+
branches: ["**"]
5+
pull_request:
6+
branches: [master]
7+
workflow_dispatch:
8+
jobs:
9+
spell-check:
10+
runs-on: ubuntu-latest
11+
steps:
12+
- name: Checkout Parent Repo
13+
uses: actions/checkout@v2
14+
with:
15+
ref: master
16+
repository: aws/aws-iot-device-sdk-embedded-C
17+
path: main
18+
- name: Clone This Repo
19+
uses: actions/checkout@v2
20+
with:
21+
path: ./kernel
22+
- name: Install spell
23+
run: |
24+
sudo apt-get install spell
25+
sudo apt-get install util-linux
26+
- name: Check spelling
27+
run: |
28+
PATH=$PATH:main/tools/spell
29+
# Make sure that the portable directory is not included in the spellcheck.
30+
sed -i 's/find $DIRNAME/find $DIRNAME -not -path '*portable*'/g' main/tools/spell/find-unknown-comment-words
31+
find-unknown-comment-words --directory kernel/
32+
if [ "$?" = "0" ]; then
33+
exit 0
34+
else
35+
exit 1
36+
fi
37+

README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,3 +33,7 @@ See the readme file in the ```./portable``` directory for more information.
3333
### Code Formatting
3434
FreeRTOS files are formatted using the "uncrustify" tool. The configuration file used by uncrustify can be found in the [FreeRTOS/FreeRTOS repository](https://github.com/FreeRTOS/FreeRTOS/blob/master/tools/uncrustify.cfg).
3535

36+
### Spelling
37+
*lexicon.txt* contains words that are not traditionally found in an English dictionary. It is used by the spellchecker to verify the various jargon, variable names, and other odd words used in the FreeRTOS code base. If your pull request fails to pass the spelling and you believe this is a mistake, then add the word to *lexicon.txt*.
38+
Note that only the FreeRTOS Kernel source files are checked for proper spelling, the portable section is ignored.
39+

include/event_groups.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -670,7 +670,7 @@ EventBits_t xEventGroupSetBits( EventGroupHandle_t xEventGroup,
670670
* xEventGroupSync( xEventBits, TASK_1_BIT, ALL_SYNC_BITS, portMAX_DELAY );
671671
*
672672
* // xEventGroupSync() was called with an indefinite block time, so
673-
* // this task will only reach here if the syncrhonisation was made by all
673+
* // this task will only reach here if the synchronisation was made by all
674674
* // three tasks, so there is no need to test the return value.
675675
* }
676676
* }
@@ -689,7 +689,7 @@ EventBits_t xEventGroupSetBits( EventGroupHandle_t xEventGroup,
689689
* xEventGroupSync( xEventBits, TASK_2_BIT, ALL_SYNC_BITS, portMAX_DELAY );
690690
*
691691
* // xEventGroupSync() was called with an indefinite block time, so
692-
* // this task will only reach here if the syncrhonisation was made by all
692+
* // this task will only reach here if the synchronisation was made by all
693693
* // three tasks, so there is no need to test the return value.
694694
* }
695695
* }

include/queue.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1535,7 +1535,7 @@ BaseType_t xQueueGiveMutexRecursive( QueueHandle_t xMutex ) PRIVILEGED_FUNCTION;
15351535
#endif
15361536

15371537
/*
1538-
* Generic version of the function used to creaet a queue using dynamic memory
1538+
* Generic version of the function used to create a queue using dynamic memory
15391539
* allocation. This is called by other functions and macros that create other
15401540
* RTOS objects that use the queue structure as their base.
15411541
*/
@@ -1546,7 +1546,7 @@ BaseType_t xQueueGiveMutexRecursive( QueueHandle_t xMutex ) PRIVILEGED_FUNCTION;
15461546
#endif
15471547

15481548
/*
1549-
* Generic version of the function used to creaet a queue using dynamic memory
1549+
* Generic version of the function used to create a queue using dynamic memory
15501550
* allocation. This is called by other functions and macros that create other
15511551
* RTOS objects that use the queue structure as their base.
15521552
*/

include/task.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3023,7 +3023,7 @@ eSleepModeStatus eTaskConfirmSleepModeStatus( void ) PRIVILEGED_FUNCTION;
30233023
TaskHandle_t pvTaskIncrementMutexHeldCount( void ) PRIVILEGED_FUNCTION;
30243024

30253025
/*
3026-
* For internal use only. Same as vTaskSetTimeOutState(), but without a critial
3026+
* For internal use only. Same as vTaskSetTimeOutState(), but without a critical
30273027
* section.
30283028
*/
30293029
void vTaskInternalSetTimeOutState( TimeOut_t * const pxTimeOut ) PRIVILEGED_FUNCTION;

0 commit comments

Comments
 (0)