View Change

3 comments:

• File atapromise.c:

  • Patch Set #5, Line 145: msg_pwarn(

    Not sure why clang-format aligned it this way... Maybe BreakStringLiterals is needed? But in this case the '\n' embedded in the string itself might show up in a weird place.

• File board_enable.c:

  • Patch Set #4, Line 305: msg_pdbg(

    More instances of weird alignment of strings as arguments to print functions.

• File ich_descriptors.h:

  • Patch Set #4, Line 118: uint32_t : 17, freq_read : 3, fastread : 1, freq_fastread : 3, freq_write : 3,

    AlignConsecutiveBitFields might work in these cases?

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