commit 32f827f9ddc7fdb1264e7bea0ed52b3fecc2826d
author mukesh agrawal <quiche@chromium.org> Tue Jul 02 13:00:01 2013 -0700
committer ChromeBot <chrome-bot@google.com> Tue Jul 02 16:31:25 2013 -0700
tree b4208056f4a764b856dd5ee8f34379be438a7f24
parent 4c9f5766056768e76c5c374e7e7878c6b932efd5

crash_collector: pedantically document hashed string for warn_collector

When computing a hash of a kernel warning message, we include the
trailing newline character. Document this fact, to make it easier
to write separate tools that compute the hash.

BUG=none
TEST=cros_workon_make --board=link --test --reconf crash-reporter

Change-Id: I5d78235db5c8bff7e3639c9d1cd3915d343f471e
Reviewed-on: https://gerrit.chromium.org/gerrit/60797
Commit-Queue: mukesh agrawal <quiche@chromium.org>
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>

crash_reporter/warn_collector.l

diff --git a/crash_reporter/warn_collector.l b/crash_reporter/warn_collector.l
index ee3657e..6fc2f56 100644
--- a/crash_reporter/warn_collector.l
+++ b/crash_reporter/warn_collector.l
@@ -48,7 +48,9 @@
 
 ^{CUT_HERE}\n{WARNING}          BEGIN(PRE_WARN);
 .|\n                            /* ignore all other input in state 0 */
-<PRE_WARN>[^ ]+.[^ ]+\n         if (WarnStart()) {  /* yytext is file:line func+offset/offset() */
+<PRE_WARN>[^ ]+.[^ ]+\n         if (WarnStart()) {
+                                  /* yytext is
+                                     "file:line func+offset/offset()\n" */
                                   BEGIN(WARN); ECHO;
                                 } else {
                                   BEGIN(0);