Patch Set 1: Code-Review-1

Thanks for doing this. There are a lot more instances of this, though... I see at least 212 instances using `grep -rH '\/\/[[:space:]]' *.c *.h | wc -l`.

Want to try and fix some more instances while you're at it? Also, maybe we can look into git hooks to prevent more C++ style comments from creeping in?


I don't mind committing this as an incremental step, though. (marking as -1 just so I see it as reviewed in the gerrit dashboard)

I know there are many more instances. I handled the files that had just a few outliers. Is there any rule about using C++ style comments, though? There's some files that make heavy use of them.

Also, I didn't touch FIXME comments on some files. They are FIXMEs, and should probably be fixed someday. (they were also at the end of a line, and using C-style comments might have made the lines a tad bit too long)

View Change

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

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I74102108ef4b0365a9bf0fea20f012aece201892
Gerrit-Change-Number: 38660
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Sat, 01 Feb 2020 22:29:07 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment