Attention is currently required from: Angel Pons, Anastasia Klimchuk.

View Change

1 comment:

• Patchset:

  • Patch Set #4:

    Yes, I was also thinking about whether io_mock is the right header ... so probably it's not.
    include/test.h gets LOG_ME further in this chain anyway CB:57914, so I can move non zero and not null there as well.
    But I would need include/tests.c for not_null() implementation, is that correct?

    You mean to put not_null() into `tests/tests.c`?

    
    Another thing I wanted to ask before I start changing everything: I am using io_mock.c later in this chain for current_io CB:57915 is that correct approach? If yes, I think I can remove io_mock.c from this patch and delay its creation until CB:57915.

    Sounds good.

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