Make all parsers return null.

When a parser receives an empty input ("".equals(input.trim())), then
return null, not new IItem() with no input.  If there is input but
invalid, a non-null is still returned.

This fixes the case in bugreports where the section is listed but there
is no input.  For example:

    ----- SECTION 1 ------

    ----- SECTION 2 ------
    text

In this case, the section 1 parser would return null.

Change-Id: Ifdfabc3bc40aa2303fcbc11ec6ccbf63167d54ae

1 files changed, 6 insertions, 3 deletions

diff --git a/src/com/android/loganalysis/parser/AbstractSectionParser.java b/src/com/android/loganalysis/parser/AbstractSectionParser.java
index 91baeaa..57974fb 100644
--- a/src/com/android/loganalysis/parser/AbstractSectionParser.java
+++ b/src/com/android/loganalysis/parser/AbstractSectionParser.java
@@ -72,10 +72,7 @@ public abstract class AbstractSectionParser implements IParser {
             }
         } else {
             runCurrentParser();
-            mParseBlock.clear();
             mCurrentParser = nextParser;
-
-            onSwitchParser();
         }
     }
 
@@ -105,6 +102,9 @@ public abstract class AbstractSectionParser implements IParser {
         mCurrentParser = parser;
     }
 
+    /**
+     * Callback for when parsers are switched.
+     */
     protected void onSwitchParser() {
     }
 
@@ -119,6 +119,9 @@ public abstract class AbstractSectionParser implements IParser {
                 // CLog.v("Just ran the %s parser", mCurrentParser.getClass().getSimpleName());
             }
         }
+
+        mParseBlock.clear();
+        onSwitchParser();
     }
 }