mirror of
https://github.com/servo/servo.git
synced 2025-07-23 07:13:52 +01:00
Support default values for restricted float members (fixes #7217)
This commit is contained in:
parent
a1b3f477aa
commit
ec2a6b24b1
3 changed files with 15 additions and 14 deletions
|
@ -97,8 +97,8 @@ numericTags = [
|
|||
IDLType.Tags.int16, IDLType.Tags.uint16,
|
||||
IDLType.Tags.int32, IDLType.Tags.uint32,
|
||||
IDLType.Tags.int64, IDLType.Tags.uint64,
|
||||
IDLType.Tags.unrestricted_float, IDLType.Tags.float,
|
||||
IDLType.Tags.unrestricted_double, IDLType.Tags.double
|
||||
IDLType.Tags.unrestricted_float,
|
||||
IDLType.Tags.unrestricted_double
|
||||
]
|
||||
|
||||
|
||||
|
@ -1056,7 +1056,9 @@ def getJSToNativeConversionInfo(type, descriptorProvider, failureCode=None,
|
|||
defaultStr = "None"
|
||||
else:
|
||||
tag = defaultValue.type.tag()
|
||||
if tag in numericTags:
|
||||
if tag in [IDLType.Tags.float, IDLType.Tags.double]:
|
||||
defaultStr = "Finite::wrap(%s)" % defaultValue.value
|
||||
elif tag in numericTags:
|
||||
defaultStr = str(defaultValue.value)
|
||||
else:
|
||||
assert(tag == IDLType.Tags.bool)
|
||||
|
|
|
@ -2920,8 +2920,7 @@ class IDLValue(IDLObject):
|
|||
elif self.type.isInteger() and type.isFloat():
|
||||
# Convert an integer literal into float
|
||||
if -2**24 <= self.value <= 2**24:
|
||||
floatType = BuiltinTypes[IDLBuiltinType.Types.float]
|
||||
return IDLValue(self.location, floatType, float(self.value))
|
||||
return IDLValue(self.location, type, float(self.value))
|
||||
else:
|
||||
raise WebIDLError("Converting value %s to %s will lose precision." %
|
||||
(self.value, type), [location])
|
||||
|
@ -2939,7 +2938,7 @@ class IDLValue(IDLObject):
|
|||
math.isnan(self.value))):
|
||||
raise WebIDLError("Trying to convert unrestricted value %s to non-unrestricted"
|
||||
% self.value, [location]);
|
||||
return self
|
||||
return IDLValue(self.location, type, self.value)
|
||||
elif self.type.isString() and type.isUSVString():
|
||||
# Allow USVStrings to use default value just like
|
||||
# DOMString. No coercion is required in this case as Codegen.py
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue