Bug 1060988 - Fix more bad implicit constructors in JS IPC code; r=billm

This commit is contained in:
Ehsan Akhgari 2014-08-31 21:10:16 -04:00
parent e51f95037f
commit ddca1158ca
7 changed files with 9 additions and 9 deletions

View File

@ -22,7 +22,7 @@ class JavaScriptBase : public WrapperOwner, public WrapperAnswer, public Base
typedef WrapperAnswer Answer;
public:
JavaScriptBase(JSRuntime *rt)
explicit JavaScriptBase(JSRuntime *rt)
: JavaScriptShared(rt),
WrapperOwner(rt),
WrapperAnswer(rt)

View File

@ -17,7 +17,7 @@ namespace jsipc {
class JavaScriptChild : public JavaScriptBase<PJavaScriptChild>
{
public:
JavaScriptChild(JSRuntime *rt);
explicit JavaScriptChild(JSRuntime *rt);
virtual ~JavaScriptChild();
bool init();

View File

@ -34,19 +34,19 @@ namespace jsipc {
struct ReceiverObj
{
ObjectId id;
ReceiverObj(ObjectId id) : id(id) {}
explicit ReceiverObj(ObjectId id) : id(id) {}
};
struct InVariant
{
JSVariant variant;
InVariant(const JSVariant &variant) : variant(variant) {}
explicit InVariant(const JSVariant &variant) : variant(variant) {}
};
struct OutVariant
{
JSVariant variant;
OutVariant(const JSVariant &variant) : variant(variant) {}
explicit OutVariant(const JSVariant &variant) : variant(variant) {}
};
class Logging

View File

@ -17,7 +17,7 @@ namespace jsipc {
class JavaScriptParent : public JavaScriptBase<PJavaScriptParent>
{
public:
JavaScriptParent(JSRuntime *rt);
explicit JavaScriptParent(JSRuntime *rt);
virtual ~JavaScriptParent();
bool init();

View File

@ -86,7 +86,7 @@ class Logging;
class JavaScriptShared
{
public:
JavaScriptShared(JSRuntime *rt);
explicit JavaScriptShared(JSRuntime *rt);
virtual ~JavaScriptShared() {}
bool init();

View File

@ -16,7 +16,7 @@ namespace jsipc {
class WrapperAnswer : public virtual JavaScriptShared
{
public:
WrapperAnswer(JSRuntime *rt) : JavaScriptShared(rt) {}
explicit WrapperAnswer(JSRuntime *rt) : JavaScriptShared(rt) {}
bool AnswerPreventExtensions(const ObjectId &objId, ReturnStatus *rs);
bool AnswerGetPropertyDescriptor(const ObjectId &objId, const nsString &id,

View File

@ -27,7 +27,7 @@ class WrapperOwner : public virtual JavaScriptShared
mozilla::ipc::IProtocol>::ActorDestroyReason
ActorDestroyReason;
WrapperOwner(JSRuntime *rt);
explicit WrapperOwner(JSRuntime *rt);
bool init();
// Fundamental proxy traps. These are required.