Attention is currently required from: Felix Singer, Thomas Heijligen, Anastasia Klimchuk.

Patch set 6:Code-Review +2

View Change

1 comment:

• File platform/endian_big.c:

  • Patch Set #4, Line 18: #include "platform.h"

    Then this should be done for all includes. […]

    Definition no, but sometimes it matters. #include "" looks in the local
    directory of the compilation unit first, then in include paths. So for
    "../platform.h" the quote form was needed.

    Using <> when "" would work too, is probably a personal preference. Why
    let the compiler look in the local dir when we know it won't find anything?

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