LGTM
Still if the protobuf generate a line of code in which "com.google.protobuf" or "hbase.protobuf.generated" is not coming and line length >100, it might show us wrong report. But normally this wont happen I guess.. With this patch we are in better position. +1 for commit.

Anoop Sam John
added a comment - 25/Mar/14 13:41 LGTM
Still if the protobuf generate a line of code in which "com.google.protobuf" or "hbase.protobuf.generated" is not coming and line length >100, it might show us wrong report. But normally this wont happen I guess.. With this patch we are in better position. +1 for commit.