Get rid of most instance of 'foo != NULL'

core/src/fpdfapi/fpdf_font/ttgsubtable.h

Index: core/src/fpdfapi/fpdf_font/ttgsubtable.h
diff --git a/core/src/fpdfapi/fpdf_font/ttgsubtable.h b/core/src/fpdfapi/fpdf_font/ttgsubtable.h
index e3c996565460789a204d6844e03785a0f8c03fdf..03bf6b3ebf252fd79a121a6e94897843906ce5eb 100644
--- a/core/src/fpdfapi/fpdf_font/ttgsubtable.h
+++ b/core/src/fpdfapi/fpdf_font/ttgsubtable.h
@@ -277,7 +277,7 @@ class CFX_CTTGSUBTable {
     TLookup()
         : LookupType(0), LookupFlag(0), SubTableCount(0), SubTable(NULL) {}
     ~TLookup() {
-      if (SubTableCount > 0 && SubTable != NULL) {
+      if (SubTableCount > 0 && SubTable) {

[Tom Sepez, 2015/12/14 19:14:07]

nit: could we write 
   if (SubTable && SubTableCount > 0)

I don't know why, but it reads much easier to me that way ...
And looking at this, I think its wrong, since we want to delete[] subtable even
if we have a subtablecout of 0?  And then the for loop guards deleting elts in
the 0 length case ... and ...  so

if (SubTable) {
  for (...)
    delete ...;
  delete ...;
}

[Lei Zhang, 2015/12/15 01:38:32]

Done.

         for (int i = 0; i < SubTableCount; i++) {
           delete SubTable[i];
         }