Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c6baf563 authored by Chih-Hung Hsieh's avatar Chih-Hung Hsieh
Browse files

Fix google-explicit-constructor warnings. 

Bug: 28341362
Change-Id: Ibdd6a210bb7ff228e3624cc319169f77aca3b51e
parent cd4f7e12

core/jni/android/graphics/Bitmap.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -380,7 +380,7 @@ using namespace android; 
// on the caller already having a local JNI ref

// on the caller already having a local JNI ref

class LocalScopedBitmap {

class LocalScopedBitmap {

public:

public:

    LocalScopedBitmap(jlong bitmapHandle)

    explicit LocalScopedBitmap(jlong bitmapHandle)

            : mBitmap(reinterpret_cast<Bitmap*>(bitmapHandle)) {}

            : mBitmap(reinterpret_cast<Bitmap*>(bitmapHandle)) {}





    Bitmap* operator->() {

    Bitmap* operator->() {

core/jni/android/graphics/HarfBuzzNGFaceSkia.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -50,7 +50,7 @@ static const bool kDebugGlyphs = false; 
// calls. See the Harfbuzz source for references about what these callbacks do.

// calls. See the Harfbuzz source for references about what these callbacks do.





struct HarfBuzzFontData {

struct HarfBuzzFontData {

    HarfBuzzFontData(SkPaint* paint) : m_paint(paint) { }

    explicit HarfBuzzFontData(SkPaint* paint) : m_paint(paint) { }

    SkPaint* m_paint;

    SkPaint* m_paint;

};

};

core/jni/android/graphics/Region.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -268,7 +268,7 @@ struct RgnIterPair { 
    SkRegion            fRgn;   // a copy of the caller's region

    SkRegion            fRgn;   // a copy of the caller's region

    SkRegion::Iterator  fIter;  // an iterator acting upon the copy (fRgn)

    SkRegion::Iterator  fIter;  // an iterator acting upon the copy (fRgn)





    RgnIterPair(const SkRegion& rgn) : fRgn(rgn) {

    explicit RgnIterPair(const SkRegion& rgn) : fRgn(rgn) {

        // have our iterator reference our copy (fRgn), so we know it will be

        // have our iterator reference our copy (fRgn), so we know it will be

        // unchanged for the lifetime of the iterator

        // unchanged for the lifetime of the iterator

        fIter.reset(fRgn);

        fIter.reset(fRgn);

core/jni/android_hardware_camera2_legacy_PerfMeasurement.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -114,7 +114,7 @@ class PerfMeasurementContext { 
     * will be active at once, which is a function of the GPU's level of

     * will be active at once, which is a function of the GPU's level of

     * pipelining and the frequency of queries.

     * pipelining and the frequency of queries.

     */

     */

    PerfMeasurementContext(size_t maxQueryCount):

    explicit PerfMeasurementContext(size_t maxQueryCount):

            mTimingStartIndex(0),

            mTimingStartIndex(0),

            mTimingEndIndex(0),

            mTimingEndIndex(0),

            mTimingQueryIndex(0) {

            mTimingQueryIndex(0) {

core/jni/android_util_AssetManager.cpp

+1 −1
Original line number Original line Diff line number Diff line
@@ -1059,7 +1059,7 @@ static void android_content_AssetManager_dumpTheme(JNIEnv* env, jobject clazz, 




class XmlAttributeFinder : public BackTrackingAttributeFinder<XmlAttributeFinder, jsize> {

class XmlAttributeFinder : public BackTrackingAttributeFinder<XmlAttributeFinder, jsize> {

public:

public:

    XmlAttributeFinder(const ResXMLParser* parser)

    explicit XmlAttributeFinder(const ResXMLParser* parser)

        : BackTrackingAttributeFinder(0, parser != NULL ? parser->getAttributeCount() : 0)

        : BackTrackingAttributeFinder(0, parser != NULL ? parser->getAttributeCount() : 0)

        , mParser(parser) {}

        , mParser(parser) {}