diff options
author | Jay Berkenbilt <ejb@ql.org> | 2022-02-04 22:31:31 +0100 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2022-02-04 22:31:31 +0100 |
commit | 2229e37e88a8d497b4a04725a46d181326a3fa45 (patch) | |
tree | 9b9c614458cc309145988c485d396f9bca172d44 /README-maintainer | |
parent | 2abb305565d1c42e60b6534953abad24cce31879 (diff) | |
download | qpdf-2229e37e88a8d497b4a04725a46d181326a3fa45.tar.zst |
Add a blank line after the first header included in each source
Diffstat (limited to 'README-maintainer')
-rw-r--r-- | README-maintainer | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/README-maintainer b/README-maintainer index 49dc643f..b43f183b 100644 --- a/README-maintainer +++ b/README-maintainer @@ -86,6 +86,15 @@ GOOGLE OSS-FUZZ CODING RULES +* In a source file, include the header file that declares the source + class first followed by a blank line. If a config file is needed + first, put a blank line between that and the header followed by + another blank line. This assures that each header file is included + first at least once, thereby ensuring that it explicitly includes + all the headers it needs, which in turn alleviates lots of header + ordering problems. The blank line ensures that formatters don't + messt his up by resorting the headers. + * Avoid atoi. Use QUtil::string_to_int instead. It does overflow/underflow checking. |