NRF_LOG_STR_PUSH_BUFFER_SIZE defined twice

[gnf]

In SDK 15.1.0 NRF_LOG_STR_PUSH_BUFFER_SIZE is defined twice in every sdk_config.h in the SDK. For example in the ble_app_template/pca10040/s132/config/sdk_config.h:

// <o> NRF_LOG_STR_PUSH_BUFFER_SIZE  - Size of the buffer dedicated for strings stored using @ref NRF_LOG_PUSH.
 
// <16=> 16 
// <32=> 32 
// <64=> 64 
// <128=> 128 
// <256=> 256 
// <512=> 512 
// <1024=> 1024 

#ifndef NRF_LOG_STR_PUSH_BUFFER_SIZE
#define NRF_LOG_STR_PUSH_BUFFER_SIZE 128
#endif

// <o> NRF_LOG_STR_PUSH_BUFFER_SIZE  - Size of the buffer dedicated for strings stored using @ref NRF_LOG_PUSH.
 
// <16=> 16 
// <32=> 32 
// <64=> 64 
// <128=> 128 
// <256=> 256 
// <512=> 512 
// <1024=> 1024 

#ifndef NRF_LOG_STR_PUSH_BUFFER_SIZE
#define NRF_LOG_STR_PUSH_BUFFER_SIZE 128
#endif

[tesc]

Hi,

Thank you for notifying us about this bug. It is highly appreciated! I have registered it in our internal tracker. Hopefully we can fix it before too long.

Regards,
Terje

[Christian Waldmann]

Now the section isn't present anymore in the released nRF5_SDK_15.2.0_9412b96

When sdding it once, it compiles

[gnf]

No. I'm sorry but I just downloaded and tested nRF5_SDK_15.2.0_9412b96. The problem is still there!

[tesc]

Hi,

I am afraid nRF5 SDK v15.2.0 was too far in the release pipeline for going back to change this before release. (This is not really a critical bug, although it may be a bit annoying.) Hopefully we will look into it shortly and fix it for future releases.

Regards,
Terje

[tesc]

Hi,

We have had some issues with some sections of the sdk_config.h file missing in some examples (but still it was present in others). This is a known issue, and is probably the reason why you did not find it in the example that you were looking at. (It is not limited to NRF_LOG_STR_PUSH_BUFFER_SIZE.)

Regards,
Terje