From: digit@google.com Date: Tue, 10 Jan 2012 10:00:59 +0000 (+0000) Subject: remove one un-needed static initializer X-Git-Tag: submit/tizen/20180928.044319~17017 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3e05f291167a265ae749b4dd9d13ec0407161082;p=platform%2Fupstream%2FlibSkiaSharp.git remove one un-needed static initializer This patch removes the use of a static C++ object in the implementation of SkMatrix44::isIdentity(). Instead, we rely on direct comparison with a statically allocated array of SkMScalar values, which is completely equivalent. Review URL: http://codereview.appspot.com/5502067 git-svn-id: http://skia.googlecode.com/svn/trunk@2991 2bbb7eff-a529-9590-31e7-b0007b416f81 --- diff --git a/src/utils/SkMatrix44.cpp b/src/utils/SkMatrix44.cpp index a59c91b60d..f00e399191 100644 --- a/src/utils/SkMatrix44.cpp +++ b/src/utils/SkMatrix44.cpp @@ -84,10 +84,14 @@ void SkMatrix44::asRowMajord(double dst[]) const { /////////////////////////////////////////////////////////////////////////////// -static const SkMatrix44 gIdentity44; - bool SkMatrix44::isIdentity() const { - return *this == gIdentity44; + static const SkMScalar sIdentityMat[4][4] = { + { 1, 0, 0, 0 }, + { 0, 1, 0, 0 }, + { 0, 0, 1, 0 }, + { 0, 0, 0, 1 }, + }; + return !memcmp(fMat, sIdentityMat, sizeof(fMat)); } ///////////////////////////////////////////////////////////////////////////////