diff options
| author | Jeremy Bettis <jbettis@google.com> | 2022-12-02 11:33:07 -0700 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-12-02 20:21:24 +0000 |
| commit | 8fd9141a45c99e51da4a373f75d75c9a70f99b43 (patch) | |
| tree | 65e35e791e204cb5599672905a4a79c9341fb118 | |
| parent | 15e1e6f666f714f0a617d5c19521901589d4582f (diff) | |
| download | chrome-ec-8fd9141a45c99e51da4a373f75d75c9a70f99b43.tar.gz | |
ec: Add #line directive to common.h
Previously #line was added to config to fix off-by-one errors in
the coverage reports. This came up again, and this time adding the #line
directive to common.h fixed it.
I hope I can find the root cause eventually to stop playing wack-a-mole.
BRANCH=None
BUG=b:260765255,b:258344465
TEST=Ran build_firmware.py with coverage
Change-Id: Ic32bbb34db441ce4fa6c52370ea44eb925cd2d1c
Signed-off-by: Jeremy Bettis <jbettis@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4076150
Commit-Queue: Tristan Honscheid <honscheid@google.com>
Tested-by: Jeremy Bettis <jbettis@chromium.org>
Reviewed-by: Tristan Honscheid <honscheid@google.com>
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
Auto-Submit: Jeremy Bettis <jbettis@chromium.org>
| -rw-r--r-- | include/common.h | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/include/common.h b/include/common.h index 02f7449d81..496fa42247 100644 --- a/include/common.h +++ b/include/common.h @@ -8,6 +8,13 @@ #ifndef __CROS_EC_COMMON_H #define __CROS_EC_COMMON_H +/* + * I don't know why but gcc's preprocessor doesn't like the autoconf.h file, + * sometimes. Adding a #line directive anywhere in this file seems to fix the + * problem. #line marks the *next* line, so it is off by one. + */ +#line 17 + #include "compile_time_macros.h" #include <inttypes.h> |