Bug 1487082 - Resolve potential for build bustage regarding bad implicit conversion constructors

* Part 1: make conversion CTORs explicit.
  Minor tweaks to conform with new(ish) static analysis rule which flags up implicit single-argument conversion constructors.
* Part 2: fix a couple of double-mRefCnt declaration errors.
* Part 3: Fix unsafe use of NS_ConvertASCIItoUTF16().

Tag #1273

1 files changed, 3 insertions, 3 deletions

diff --git a/db/mork/src/morkDeque.h b/db/mork/src/morkDeque.h
index 4375ecf02..ad38cdbe2 100644
--- a/db/mork/src/morkDeque.h
+++ b/db/mork/src/morkDeque.h
@@ -41,9 +41,9 @@ public:
   morkNext*  mNext_Link;
   
 public:
-  morkNext(int inZero) : mNext_Link( 0 ) { }
+  explicit morkNext(int inZero) : mNext_Link( 0 ) { }
   
-  morkNext(morkNext* ioLink) : mNext_Link( ioLink ) { }
+  explicit morkNext(morkNext* ioLink) : mNext_Link( ioLink ) { }
   
   morkNext(); // mNext_Link( 0 ), { }
   
@@ -118,7 +118,7 @@ public:
   morkLink*  mLink_Prev;
   
 public:
-  morkLink(int inZero) : mLink_Next( 0 ), mLink_Prev( 0 ) { }
+  explicit morkLink(int inZero) : mLink_Next( 0 ), mLink_Prev( 0 ) { }
   
   morkLink(); // mLink_Next( 0 ), mLink_Prev( 0 ) { }