Attention is currently required from: Nico Huber, Michał Żygowski, Jonathan Zhang, David Hendricks.

View Change

1 comment:

• File ich_descriptors.c:

  • Patch Set #1, Line 968:

    		if (content->ICCRIBA != 0x34)
    			msg_pwarn("Unknown flash descriptor, assuming 300 series compatibility.\n");
    

    "Unknown flash descriptor, assuming 300 series compatibility" and "Peculiar flash descriptor, assumi […]

    AIUI, the message in this path says "Unknown" because it's the path where none of the evaluated conditions is true. In the "Peculiar" paths, we know more about the flash descriptor because at least one check is true.

    If one were to remove all instances of `else` in this function (while preserving behavior, of course), the "Unknown" message would be at the end of the function, whereas the "Peculiar" messages would all be inside an if-block. Hmmm, I like the refactoring idea.

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