• Mark OLESEN's avatar
    STYLE: simplify short-circuit involving autoPtr (#1775) · 9af3f85c
    Mark OLESEN authored
    - with '&&' conditions, often better to check for non-null autoPtr
      first (it is cheap)
    - check as bool instead of valid() method for cleaner code, especially
      when the wrapped item itself has a valid/empty or good.
      Also when handling multiple checks.
          if (ptr && ptr->valid())
          if (ptr1 || ptr2)
          if (ptr.valid() && ptr->valid())
          if (ptr1.valid() || ptr2.valid())