WICED Studio Wi-Fi Combo

[grphc_1578046]

In wwd_assert.c there is a bug the will cause the system to fail to build if the DEBUG and WPRINT_ENABLE_ERROR is enabled.

There are missing braces around the WPRINT_MACRO - current source is:

#define wiced_minor_assert( error_string, assertion )   do { if ( !(assertion) ) WPRINT_MACRO( error_string ); } while(0)

corrected source is:

#define wiced_minor_assert( error_string, assertion )   do { if ( !(assertion) ) WPRINT_MACRO( ( error_string ) ); } while(0)

This is on line 56 - Please correct for next release.

[GauravS_31 · Accepted solution]

Thank you for reporting this bug. I could reproduce the build error and the fix corrected the problem. I have raised a ticket and reported this bug internally and this should be corrected.

[AxLi_1746341]

grsr wrote:

Thank you for reporting this bug. I could reproduce the build error and the fix corrected the problem. I have raised a ticket and reported this bug internally and this should be corrected.

wiced-studio-5.0 does not include this fix.

I'm wondering why the fix keep getting lost in new sdk?

Seem we are keep wasting time in your internal communication.

[grphc_1578046]

STILL NOT in 5.1 release - WHAT DOES IT TAKE?

[GregG_16]

Let us check what release this will make it into.

[AxLi_1746341]

I just cannot believe such trivial fix needs more than 6 months to merge.

Looking at the discussion thread, the time is mostly spent on message passing.

That discourages people to contribute fixes.

[saching_06]

We appreciate your efforts in reporting this bug as well as a fix. Looks like it is fixed in WICED Studio 5.2. Here is the code from latest  wwd_assert.h file:

#define wiced_minor_assert( error_string, assertion ) do { if ( !(assertion) ) WPRINT_MACRO( ( error_string ) ); } while(0)

I agree it should not have taken so long to get this fix included in WICED Studio.

Thanks for making WICED better!