Attention is currently required from: Edward O'Callaghan, Angel Pons, Nikolai Artemiev.

View Change

1 comment:

• File raiden_debug_spi.c:

  • Patch Set #2, Line 1404:

    free(ctx_data);
    	free(spi_config);
    

    Something I just thought right now: do I need to swap these two lines? (same for another branch in this function, and same for init function).
    I thought yes, and then I thought no it's not needed: spi_config is independent pointer and it won't disappear after free(ctx_data)?
    Just wanted to check that it is indeed not needed?

To view, visit change 57972. To unsubscribe, or for help writing mail filters, visit settings.