mirror of
https://github.com/servo/servo.git
synced 2025-08-04 13:10:20 +01:00
Remove the codeToSetNull argument to wrapObjectTemplate.
This commit is contained in:
parent
022dfa0a5e
commit
904e8eaf17
1 changed files with 4 additions and 6 deletions
|
@ -556,7 +556,7 @@ def getJSToNativeConversionTemplate(type, descriptorProvider, failureCode=None,
|
||||||
# A helper function for wrapping up the template body for
|
# A helper function for wrapping up the template body for
|
||||||
# possibly-nullable objecty stuff
|
# possibly-nullable objecty stuff
|
||||||
def wrapObjectTemplate(templateBody, isDefinitelyObject, type,
|
def wrapObjectTemplate(templateBody, isDefinitelyObject, type,
|
||||||
codeToSetNull, failureCode=None):
|
failureCode=None):
|
||||||
if not isDefinitelyObject:
|
if not isDefinitelyObject:
|
||||||
# Handle the non-object cases by wrapping up the whole
|
# Handle the non-object cases by wrapping up the whole
|
||||||
# thing in an if cascade.
|
# thing in an if cascade.
|
||||||
|
@ -566,13 +566,13 @@ def getJSToNativeConversionTemplate(type, descriptorProvider, failureCode=None,
|
||||||
if type.nullable():
|
if type.nullable():
|
||||||
templateBody += (
|
templateBody += (
|
||||||
"} else if (${val}).is_null_or_undefined() {\n"
|
"} else if (${val}).is_null_or_undefined() {\n"
|
||||||
" %s;\n" % codeToSetNull)
|
" ${declName} = None;\n")
|
||||||
templateBody += (
|
templateBody += (
|
||||||
"} else {\n" +
|
"} else {\n" +
|
||||||
CGIndenter(onFailureNotAnObject(failureCode)).define() +
|
CGIndenter(onFailureNotAnObject(failureCode)).define() +
|
||||||
"}")
|
"}")
|
||||||
if type.nullable():
|
if type.nullable():
|
||||||
templateBody = handleDefaultNull(templateBody, codeToSetNull)
|
templateBody = handleDefaultNull(templateBody, "${declName} = None")
|
||||||
else:
|
else:
|
||||||
assert(defaultValue is None)
|
assert(defaultValue is None)
|
||||||
|
|
||||||
|
@ -625,7 +625,6 @@ def getJSToNativeConversionTemplate(type, descriptorProvider, failureCode=None,
|
||||||
conversion = ("${declName} = Some(%s::new((${val}).to_object()));" % name)
|
conversion = ("${declName} = Some(%s::new((${val}).to_object()));" % name)
|
||||||
|
|
||||||
template = wrapObjectTemplate(conversion, isDefinitelyObject, type,
|
template = wrapObjectTemplate(conversion, isDefinitelyObject, type,
|
||||||
"${declName} = None",
|
|
||||||
failureCode)
|
failureCode)
|
||||||
return (template, declType, None, isOptional, None)
|
return (template, declType, None, isOptional, None)
|
||||||
|
|
||||||
|
@ -667,8 +666,7 @@ def getJSToNativeConversionTemplate(type, descriptorProvider, failureCode=None,
|
||||||
"}\n")
|
"}\n")
|
||||||
|
|
||||||
templateBody = wrapObjectTemplate(templateBody, isDefinitelyObject,
|
templateBody = wrapObjectTemplate(templateBody, isDefinitelyObject,
|
||||||
type, "${declName} = None",
|
type, failureCode)
|
||||||
failureCode)
|
|
||||||
|
|
||||||
declType = CGGeneric(typePtr)
|
declType = CGGeneric(typePtr)
|
||||||
if type.nullable() or isOptional:
|
if type.nullable() or isOptional:
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue