Attention is currently required from: Edward O'Callaghan, Angel Pons, Anastasia Klimchuk.

Patch set 4:Code-Review +1

View Change

2 comments:

• Patchset:

  • Patch Set #4:

    I wonder if `io_mock` is the right place for it. not_null() doesn't
    have to be about I/O, does it? There's an `include/test.h`, maybe
    that would be a better place?

• File tests/io_mock.h:

  • Patch Set #1, Line 38: NOT_NULL ((void *)NOT_ZERO)

    I created not_null() function, but left NON_ZERO as macro, is this what you meant, or you meant them […]

    Edward, is this what you had in mind?

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