On 10/08/2017 05:43, Martin Hosken wrote:
Dear Behdad,

-  if (buffer == NULL)
+  if (!buffer)

Always wanting to learn. How does this cause a divide by zero? Or what led you 
to make the change?

It doesn't, that's purely stylistic. The real change in this commit comes later on:

const OT::IndexSubtableRecord *subtable_record = this->cblc->find_table(glyph, &x_ppem, &y_ppem);
-    if (subtable_record == NULL)
+    if (!subtable_record || !x_ppem || !y_ppem)
       return false;

with the addition of the zero-check for the x_ppem and y_ppem values.

_______________________________________________
HarfBuzz mailing list
HarfBuzz@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/harfbuzz

Reply via email to