Skip to content

Conversation

@sarenameas
Copy link
Contributor

Description

Broker refers to the MQTT server. This term is not applicable to HTTP. This update the comments to say server instead of broker.

Test Steps

n/a

Related Issue

n/a

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sarenameas sarenameas requested a review from a team as a code owner December 4, 2020 19:39
@sarenameas sarenameas changed the title Change all occurrences of broker to server. Change all occurrences of "broker" to "server" in HTTP code comments. Dec 4, 2020
@sarenameas sarenameas merged commit 4538b99 into FreeRTOS:master Dec 4, 2020
@sarenameas sarenameas deleted the http_server branch December 4, 2020 23:09
moninom1 pushed a commit that referenced this pull request Sep 14, 2022
Prevents this warning from the TI CCS compiler:
    warning #190-D: enumerated type mixed with another type

Regardless of this compiler warning it seems like a good idea to signal
that this conversion is OK and expected. This is already being done when
doing the opposite conversion.
paulbartell pushed a commit to paulbartell/FreeRTOS that referenced this pull request Mar 30, 2023
The configuration was updated using

    uncrustify -c .github/uncrustify.cfg -o .github/uncrustify.cfg --update-config-with-doc

to align with the actually used uncrustify version used, i.e., all
configuration is now explicitly set (and no longer implicit).

The files that are common to all ports ("portable/MemMang*" and
"portable/Common/mpu_wrappers.c" are now also autoformatted.

Co-authored-by: alfred gedeon <[email protected]>
Zangetsu112 pushed a commit to Zangetsu112/FreeRTOS-evpp that referenced this pull request Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants