Dear Stefan Roese,
In message 200810091659.55186.sr@denx.de you wrote:
- else
- else {
/* Let's wait a while for the timeout to elaps.* it has no real use, but it keeps the interface happy. */wait_ms(timeout);- }
Good catch.
Quite honest, I think this is *way* to pedantic. I'd really prefer to let people who are contributing significantly do their work instead of bugging them with such rare coding style violations.
ACK from me here. "Minor" coding style violations like these braces issue or too long lines should not delay patches.
NAK, and NAK again from me. Note that it's not only a single small violation, there is a whole list of it evenin this short code snippet: - incorrect brace style - unnecessary indentation - incorrect multi-line comment style
It is much easier to fix such code when it's being added, than to fix it later (which may or may not happen). See all the examples that got commited that way - and that haven't been fixed since.
Best regards,
Wolfgang Denk