Deprecate the misnamed YOTTA_CFG_MTRACE_TMP_LINE_LENGTH flag. #43
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tommikas
changed the title
…s to be removed in the next release. Configuring the tmp line length value through config.json in yotta requires defining a separate "mtrace" group, because the name of the flag doesn't follow the 'YOTTA_CFG_MBED_TRACE_' pattern. This allows configuring the tmp line length in the same group mbed-trace is enabled in. (The name of the flag seems to be a relic of the time the name of the library was still being decided. For some reason it wasn't changed with the other ones when 'mbed-trace' was decided on.)
|
lgtm |
tommikas
pushed a commit
that referenced
this pull request
Jun 16, 2017
* origin/master: (22 commits) output/ added to gitignore list (#66) Added include_directories(${CMAKE_CURRENT_SOURCE_DIR}/) include to resolve compilation error for Linux. (#63) Ensure tr_array doesn't print <null> when len == 0 (#65) Add CMakeLists.txt to support PAL Tools (#61) Fix header documentation (#58) Update README.md (#57) Provide environment-agnostic configuration macros. (#54) Flash size can be limited by defining MBED_TRACE_MAX_LEVEL Use temp variable in mutex release loop (#52) Fix race condition when unlocking mutex Add instructions for enabling the library in mbed OS 5. (#50) document update. (#47) Allow setting mutex locking/unlocking callbacks before trace init. (#46) Ipv6 helpers update (#44) Fix disabling traces through the YOTTA_CFG_MBED_TRACE flag. (#45) Deprecate the misnamed YOTTA_CFG_MTRACE_TMP_LINE_LENGTH flag, which is to be removed in the next release. (#43) Changed dummy definitions to match the actual functions' return types (#40) FEA_TRACE_SUPPORT flag is always set when traces are enabled. (#39) Release merge to master (#38) Add nanostack-libservice lib to CMakeLists (#31) ...
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Configuring the tmp line length value through config.json in yotta requires defining a separate "mtrace" group, because the name of the flag doesn't follow the 'YOTTA_CFG_MBED_TRACE_' pattern.
This allows configuring the tmp line length in the same group mbed-trace is enabled in.
(The name of the flag seems to be a relic of the time the name of the library was still being decided.
For some reason it wasn't changed with the other ones when 'mbed-trace' was decided on.)