Bug 780108 - Get rid of the double switch in XPCConvert::JSData2Native; r=bholley

This commit is contained in:
Ms2ger 2012-08-04 09:28:45 +02:00
parent cc3b2a1e47
commit 2b7b31e7c2

View File

@ -1,4 +1,4 @@
/* -*- Mode: C++; tab-width: 8; indent-tabs-mode: nil; c-basic-offset: 4 -*- /* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
* vim: set ts=8 sw=4 et tw=78: * vim: set ts=8 sw=4 et tw=78:
* *
* This Source Code Form is subject to the terms of the Mozilla Public * This Source Code Form is subject to the terms of the Mozilla Public
@ -468,9 +468,6 @@ XPCConvert::JSData2Native(XPCCallContext& ccx, void* d, jsval s,
case nsXPTType::T_JSVAL : case nsXPTType::T_JSVAL :
*((jsval*)d) = s; *((jsval*)d) = s;
break; break;
default:
switch (type.TagPart()) {
case nsXPTType::T_VOID: case nsXPTType::T_VOID:
XPC_LOG_ERROR(("XPCConvert::JSData2Native : void* params not supported")); XPC_LOG_ERROR(("XPCConvert::JSData2Native : void* params not supported"));
NS_ERROR("void* params not supported"); NS_ERROR("void* params not supported");
@ -789,7 +786,6 @@ XPCConvert::JSData2Native(XPCCallContext& ccx, void* d, jsval s,
NS_ERROR("bad type"); NS_ERROR("bad type");
return false; return false;
} }
}
return true; return true;
} }