diff --git a/Cargo.toml b/Cargo.toml index bfd347d784..04c3bad5a5 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -16,7 +16,7 @@ members = [ [workspace.package] edition = "2021" version = "0.16.0" -rust-version = "1.64" +rust-version = "1.65" authors = ["boa-dev"] repository = "https://github.com/boa-dev/boa" license = "Unlicense/MIT" diff --git a/boa_ast/src/lib.rs b/boa_ast/src/lib.rs index 0df53eda68..c9781d2b54 100644 --- a/boa_ast/src/lib.rs +++ b/boa_ast/src/lib.rs @@ -49,11 +49,7 @@ nonstandard_style, missing_docs )] -#![allow( - clippy::module_name_repetitions, - clippy::too_many_lines, - rustdoc::missing_doc_code_examples -)] +#![allow(clippy::module_name_repetitions, clippy::too_many_lines)] mod position; mod punctuator; diff --git a/boa_engine/src/bigint.rs b/boa_engine/src/bigint.rs index 5ab9221ed1..1035801282 100644 --- a/boa_engine/src/bigint.rs +++ b/boa_engine/src/bigint.rs @@ -149,13 +149,10 @@ impl JsBigInt { #[inline] pub fn pow(x: &Self, y: &Self) -> JsResult { - let y = if let Some(y) = y.inner.to_biguint() { - y - } else { - return Err(JsNativeError::range() - .with_message("BigInt negative exponent") - .into()); - }; + let y = y + .inner + .to_biguint() + .ok_or_else(|| JsNativeError::range().with_message("BigInt negative exponent"))?; let num_bits = (x.inner.bits() as f64 * y.to_f64().expect("Unable to convert from BigUInt to f64")) diff --git a/boa_engine/src/builtins/array/mod.rs b/boa_engine/src/builtins/array/mod.rs index 7d3f6d62fc..846cec01c9 100644 --- a/boa_engine/src/builtins/array/mod.rs +++ b/boa_engine/src/builtins/array/mod.rs @@ -300,9 +300,7 @@ impl Array { /// by `Array.prototype.concat`. fn is_concat_spreadable(o: &JsValue, context: &mut Context) -> JsResult { // 1. If Type(O) is not Object, return false. - let o = if let Some(o) = o.as_object() { - o - } else { + let Some(o) = o.as_object() else { return Ok(false); }; @@ -461,9 +459,7 @@ impl Array { let next = iterator_record.step(context)?; // iv. If next is false, then - let next = if let Some(next) = next { - next - } else { + let Some(next) = next else { // 1. Perform ? Set(A, "length", 𝔽(k), true). a.set("length", k, true, context)?; diff --git a/boa_engine/src/builtins/array_buffer/mod.rs b/boa_engine/src/builtins/array_buffer/mod.rs index e11b6d0410..561fa324b8 100644 --- a/boa_engine/src/builtins/array_buffer/mod.rs +++ b/boa_engine/src/builtins/array_buffer/mod.rs @@ -148,33 +148,25 @@ impl ArrayBuffer { ) -> JsResult { // 1. Let O be the this value. // 2. Perform ? RequireInternalSlot(O, [[ArrayBufferData]]). - let obj = if let Some(obj) = this.as_object() { - obj - } else { - return Err(JsNativeError::typ() - .with_message("ArrayBuffer.byteLength called with non-object value") - .into()); - }; + let obj = this.as_object().ok_or_else(|| { + JsNativeError::typ().with_message("ArrayBuffer.byteLength called with non-object value") + })?; let obj = obj.borrow(); - let o = if let Some(o) = obj.as_array_buffer() { - o - } else { - return Err(JsNativeError::typ() - .with_message("ArrayBuffer.byteLength called with invalid object") - .into()); - }; + let buf = obj.as_array_buffer().ok_or_else(|| { + JsNativeError::typ().with_message("ArrayBuffer.byteLength called with invalid object") + })?; // TODO: Shared Array Buffer // 3. If IsSharedArrayBuffer(O) is true, throw a TypeError exception. // 4. If IsDetachedBuffer(O) is true, return +0𝔽. - if Self::is_detached_buffer(o) { + if Self::is_detached_buffer(buf) { return Ok(0.into()); } // 5. Let length be O.[[ArrayBufferByteLength]]. // 6. Return 𝔽(length). - Ok(o.array_buffer_byte_length.into()) + Ok(buf.array_buffer_byte_length.into()) } /// `25.1.5.3 ArrayBuffer.prototype.slice ( start, end )` @@ -186,34 +178,26 @@ impl ArrayBuffer { fn slice(this: &JsValue, args: &[JsValue], context: &mut Context) -> JsResult { // 1. Let O be the this value. // 2. Perform ? RequireInternalSlot(O, [[ArrayBufferData]]). - let obj = if let Some(obj) = this.as_object() { - obj - } else { - return Err(JsNativeError::typ() - .with_message("ArrayBuffer.slice called with non-object value") - .into()); - }; + let obj = this.as_object().ok_or_else(|| { + JsNativeError::typ().with_message("ArrayBuffer.slice called with non-object value") + })?; let obj_borrow = obj.borrow(); - let o = if let Some(o) = obj_borrow.as_array_buffer() { - o - } else { - return Err(JsNativeError::typ() - .with_message("ArrayBuffer.slice called with invalid object") - .into()); - }; + let buf = obj_borrow.as_array_buffer().ok_or_else(|| { + JsNativeError::typ().with_message("ArrayBuffer.slice called with invalid object") + })?; // TODO: Shared Array Buffer // 3. If IsSharedArrayBuffer(O) is true, throw a TypeError exception. // 4. If IsDetachedBuffer(O) is true, throw a TypeError exception. - if Self::is_detached_buffer(o) { + if Self::is_detached_buffer(buf) { return Err(JsNativeError::typ() .with_message("ArrayBuffer.slice called with detached buffer") .into()); } // 5. Let len be O.[[ArrayBufferByteLength]]. - let len = o.array_buffer_byte_length as i64; + let len = buf.array_buffer_byte_length as i64; // 6. Let relativeStart be ? ToIntegerOrInfinity(start). let relative_start = args.get_or_undefined(0).to_integer_or_infinity(context)?; @@ -298,14 +282,14 @@ impl ArrayBuffer { // 22. NOTE: Side-effects of the above steps may have detached O. // 23. If IsDetachedBuffer(O) is true, throw a TypeError exception. - if Self::is_detached_buffer(o) { + if Self::is_detached_buffer(buf) { return Err(JsNativeError::typ() .with_message("ArrayBuffer detached while ArrayBuffer.slice was running") .into()); } // 24. Let fromBuf be O.[[ArrayBufferData]]. - let from_buf = o + let from_buf = buf .array_buffer_data .as_ref() .expect("ArrayBuffer cannot be detached here"); @@ -389,13 +373,9 @@ impl ArrayBuffer { // 2. If IsDetachedBuffer(srcBuffer) is true, throw a TypeError exception. // 3. Let srcBlock be srcBuffer.[[ArrayBufferData]]. - let src_block = if let Some(b) = &self.array_buffer_data { - b - } else { - return Err(JsNativeError::syntax() - .with_message("Cannot clone detached array buffer") - .into()); - }; + let src_block = self.array_buffer_data.as_deref().ok_or_else(|| { + JsNativeError::syntax().with_message("Cannot clone detached array buffer") + })?; { // 4. Let targetBlock be targetBuffer.[[ArrayBufferData]]. diff --git a/boa_engine/src/builtins/date/mod.rs b/boa_engine/src/builtins/date/mod.rs index 3bd5bc6f21..f3b10f973e 100644 --- a/boa_engine/src/builtins/date/mod.rs +++ b/boa_engine/src/builtins/date/mod.rs @@ -508,13 +508,9 @@ impl Date { ) -> JsResult { // 1. Let O be the this value. // 2. If Type(O) is not Object, throw a TypeError exception. - let o = if let Some(o) = this.as_object() { - o - } else { - return Err(JsNativeError::typ() - .with_message("Date.prototype[@@toPrimitive] called on non object") - .into()); - }; + let o = this.as_object().ok_or_else(|| { + JsNativeError::typ().with_message("Date.prototype[@@toPrimitive] called on non object") + })?; let hint = args.get_or_undefined(0); @@ -908,25 +904,13 @@ impl Date { } // 3. Let y be ? ToNumber(year). - let y = args - .get(0) - .cloned() - .unwrap_or_default() - .to_number(context)?; + let y = args.get_or_undefined(0).to_number(context)?; // 4. If month is not present, let m be MonthFromTime(t); otherwise, let m be ? ToNumber(month). - let m = if let Some(m) = args.get(1) { - Some(m.to_number(context)?) - } else { - None - }; + let m = args.get(1).map(|v| v.to_number(context)).transpose()?; // 5. If date is not present, let dt be DateFromTime(t); otherwise, let dt be ? ToNumber(date). - let dt = if let Some(dt) = args.get(2) { - Some(dt.to_number(context)?) - } else { - None - }; + let dt = args.get(2).map(|v| v.to_number(context)).transpose()?; // 6. Let newDate be MakeDate(MakeDay(y, m, dt), TimeWithinDay(t)). t.set_components(false, Some(y), m, dt, None, None, None, None); @@ -965,25 +949,13 @@ impl Date { .to_number(context)?; // 3. If min is not present, let m be MinFromTime(t); otherwise, let m be ? ToNumber(min). - let m = if let Some(m) = args.get(1) { - Some(m.to_number(context)?) - } else { - None - }; + let m = args.get(1).map(|v| v.to_number(context)).transpose()?; // 4. If sec is not present, let s be SecFromTime(t); otherwise, let s be ? ToNumber(sec). - let sec = if let Some(sec) = args.get(2) { - Some(sec.to_number(context)?) - } else { - None - }; + let sec = args.get(2).map(|v| v.to_number(context)).transpose()?; // 5. If ms is not present, let milli be msFromTime(t); otherwise, let milli be ? ToNumber(ms). - let milli = if let Some(milli) = args.get(3) { - Some(milli.to_number(context)?) - } else { - None - }; + let milli = args.get(3).map(|v| v.to_number(context)).transpose()?; // 6. Let date be MakeDate(Day(t), MakeTime(h, m, s, milli)). t.set_components(false, None, None, None, Some(h), m, sec, milli); @@ -1062,18 +1034,10 @@ impl Date { .to_number(context)?; // 3. If sec is not present, let s be SecFromTime(t); otherwise, let s be ? ToNumber(sec). - let s = if let Some(s) = args.get(1) { - Some(s.to_number(context)?) - } else { - None - }; + let s = args.get(1).map(|v| v.to_number(context)).transpose()?; // 4. If ms is not present, let milli be msFromTime(t); otherwise, let milli be ? ToNumber(ms). - let milli = if let Some(milli) = args.get(2) { - Some(milli.to_number(context)?) - } else { - None - }; + let milli = args.get(2).map(|v| v.to_number(context)).transpose()?; // 5. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), m, s, milli)). t.set_components(false, None, None, None, None, Some(m), s, milli); @@ -1110,11 +1074,7 @@ impl Date { .to_number(context)?; // 3. If date is not present, let dt be DateFromTime(t); otherwise, let dt be ? ToNumber(date). - let dt = if let Some(date) = args.get(1) { - Some(date.to_number(context)?) - } else { - None - }; + let dt = args.get(1).map(|v| v.to_number(context)).transpose()?; // 4. Let newDate be MakeDate(MakeDay(YearFromTime(t), m, dt), TimeWithinDay(t)). t.set_components(false, None, Some(m), dt, None, None, None, None); @@ -1155,11 +1115,7 @@ impl Date { .to_number(context)?; // 3. If ms is not present, let milli be msFromTime(t); otherwise, let milli be ? ToNumber(ms). - let milli = if let Some(milli) = args.get(1) { - Some(milli.to_number(context)?) - } else { - None - }; + let milli = args.get(1).map(|v| v.to_number(context)).transpose()?; // 4. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), MinFromTime(t), s, milli)). t.set_components(false, None, None, None, None, None, Some(s), milli); @@ -1341,18 +1297,10 @@ impl Date { .to_number(context)?; // 4. If month is not present, let m be MonthFromTime(t); otherwise, let m be ? ToNumber(month). - let m = if let Some(m) = args.get(1) { - Some(m.to_number(context)?) - } else { - None - }; + let m = args.get(1).map(|v| v.to_number(context)).transpose()?; // 5. If date is not present, let dt be DateFromTime(t); otherwise, let dt be ? ToNumber(date). - let dt = if let Some(dt) = args.get(2) { - Some(dt.to_number(context)?) - } else { - None - }; + let dt = args.get(2).map(|v| v.to_number(context)).transpose()?; // 6. Let newDate be MakeDate(MakeDay(y, m, dt), TimeWithinDay(t)). t.set_components(true, Some(y), m, dt, None, None, None, None); @@ -1395,25 +1343,13 @@ impl Date { .to_number(context)?; // 3. If min is not present, let m be MinFromTime(t); otherwise, let m be ? ToNumber(min). - let m = if let Some(m) = args.get(1) { - Some(m.to_number(context)?) - } else { - None - }; + let m = args.get(1).map(|v| v.to_number(context)).transpose()?; // 4. If sec is not present, let s be SecFromTime(t); otherwise, let s be ? ToNumber(sec). - let sec = if let Some(s) = args.get(2) { - Some(s.to_number(context)?) - } else { - None - }; + let sec = args.get(2).map(|v| v.to_number(context)).transpose()?; // 5. If ms is not present, let milli be msFromTime(t); otherwise, let milli be ? ToNumber(ms). - let ms = if let Some(ms) = args.get(3) { - Some(ms.to_number(context)?) - } else { - None - }; + let ms = args.get(3).map(|v| v.to_number(context)).transpose()?; // 6. Let newDate be MakeDate(Day(t), MakeTime(h, m, s, milli)). t.set_components(true, None, None, None, Some(h), m, sec, ms); @@ -1493,21 +1429,13 @@ impl Date { // 3. If sec is not present, let s be SecFromTime(t). // 4. Else, - let s = if let Some(s) = args.get(1) { - // a. Let s be ? ToNumber(sec). - Some(s.to_number(context)?) - } else { - None - }; + // a. Let s be ? ToNumber(sec). + let s = args.get(1).map(|v| v.to_number(context)).transpose()?; // 5. If ms is not present, let milli be msFromTime(t). // 6. Else, - let milli = if let Some(ms) = args.get(2) { - // a. Let milli be ? ToNumber(ms). - Some(ms.to_number(context)?) - } else { - None - }; + // a. Let milli be ? ToNumber(ms). + let milli = args.get(2).map(|v| v.to_number(context)).transpose()?; // 7. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), m, s, milli)). t.set_components(true, None, None, None, None, Some(m), s, milli); @@ -1549,12 +1477,8 @@ impl Date { // 3. If date is not present, let dt be DateFromTime(t). // 4. Else, - let dt = if let Some(dt) = args.get(1) { - // a. Let dt be ? ToNumber(date). - Some(dt.to_number(context)?) - } else { - None - }; + // a. Let dt be ? ToNumber(date). + let dt = args.get(1).map(|v| v.to_number(context)).transpose()?; // 5. Let newDate be MakeDate(MakeDay(YearFromTime(t), m, dt), TimeWithinDay(t)). t.set_components(true, None, Some(m), dt, None, None, None, None); @@ -1596,12 +1520,8 @@ impl Date { // 3. If ms is not present, let milli be msFromTime(t). // 4. Else, - let milli = if let Some(milli) = args.get(1) { - // a. Let milli be ? ToNumber(ms). - Some(milli.to_number(context)?) - } else { - None - }; + // a. Let milli be ? ToNumber(ms). + let milli = args.get(1).map(|v| v.to_number(context)).transpose()?; // 5. Let date be MakeDate(Day(t), MakeTime(HourFromTime(t), MinFromTime(t), s, milli)). t.set_components(true, None, None, None, None, None, Some(s), milli); @@ -1841,9 +1761,7 @@ impl Date { // This method is implementation-defined and discouraged, so we just require the same format as the string // constructor. - let date = if let Some(arg) = args.get(0) { - arg - } else { + let Some(date) = args.get(0) else { return Ok(JsValue::nan()); }; diff --git a/boa_engine/src/builtins/error/mod.rs b/boa_engine/src/builtins/error/mod.rs index 863417d779..3bfca5bdaf 100644 --- a/boa_engine/src/builtins/error/mod.rs +++ b/boa_engine/src/builtins/error/mod.rs @@ -170,14 +170,10 @@ impl Error { context: &mut Context, ) -> JsResult { // 1. Let O be the this value. - let o = if let Some(o) = this.as_object() { - o // 2. If Type(O) is not Object, throw a TypeError exception. - } else { - return Err(JsNativeError::typ() - .with_message("'this' is not an Object") - .into()); - }; + let o = this + .as_object() + .ok_or_else(|| JsNativeError::typ().with_message("'this' is not an Object"))?; // 3. Let name be ? Get(O, "name"). let name = o.get(js_string!("name"), context)?; diff --git a/boa_engine/src/builtins/eval/mod.rs b/boa_engine/src/builtins/eval/mod.rs index a7a2c287bf..ae361bfcd2 100644 --- a/boa_engine/src/builtins/eval/mod.rs +++ b/boa_engine/src/builtins/eval/mod.rs @@ -77,9 +77,7 @@ impl Eval { debug_assert!(direct || !strict); // 2. If Type(x) is not String, return x. - let x = if let Some(x) = x.as_string() { - x.clone() - } else { + let Some(x) = x.as_string() else { return Ok(x.clone()); }; diff --git a/boa_engine/src/builtins/function/mod.rs b/boa_engine/src/builtins/function/mod.rs index 0967f20a41..c414169616 100644 --- a/boa_engine/src/builtins/function/mod.rs +++ b/boa_engine/src/builtins/function/mod.rs @@ -841,15 +841,9 @@ impl BuiltInFunctionObject { } }; - let name = if let Some(name) = name { - if name.is_empty() { - "anonymous".into() - } else { - name - } - } else { - "anonymous".into() - }; + let name = name + .filter(|n| !n.is_empty()) + .unwrap_or_else(|| "anonymous".into()); match function { Function::Native { .. } | Function::Closure { .. } | Function::Ordinary { .. } => { diff --git a/boa_engine/src/builtins/iterable/async_from_sync_iterator.rs b/boa_engine/src/builtins/iterable/async_from_sync_iterator.rs index 86d9e98383..005ec571ae 100644 --- a/boa_engine/src/builtins/iterable/async_from_sync_iterator.rs +++ b/boa_engine/src/builtins/iterable/async_from_sync_iterator.rs @@ -233,10 +233,8 @@ impl AsyncFromSyncIterator { // 10. IfAbruptRejectPromise(result, promiseCapability). if_abrupt_reject_promise!(result, promise_capability, context); - // 11. If Type(result) is not Object, then - let result = if let Some(result) = result.as_object() { - result - } else { + let Some(result) = result.as_object() else { + // 11. If Type(result) is not Object, then // a. Perform ! Call(promiseCapability.[[Reject]], undefined, « a newly created TypeError object »). promise_capability .reject() @@ -333,10 +331,8 @@ impl AsyncFromSyncIterator { // 10. IfAbruptRejectPromise(result, promiseCapability). if_abrupt_reject_promise!(result, promise_capability, context); - // 11. If Type(result) is not Object, then - let result = if let Some(result) = result.as_object() { - result - } else { + let Some(result) = result.as_object() else { + // 11. If Type(result) is not Object, then // a. Perform ! Call(promiseCapability.[[Reject]], undefined, « a newly created TypeError object »). promise_capability .reject() diff --git a/boa_engine/src/builtins/iterable/mod.rs b/boa_engine/src/builtins/iterable/mod.rs index f10ea67222..789abd4405 100644 --- a/boa_engine/src/builtins/iterable/mod.rs +++ b/boa_engine/src/builtins/iterable/mod.rs @@ -353,13 +353,9 @@ impl IteratorRecord { // Note: We check if iteratorRecord.[[NextMethod]] is callable here. // This check would happen in `Call` according to the spec, but we do not implement call for `JsValue`. - let next_method = if let Some(next_method) = self.next_method.as_callable() { - next_method - } else { - return Err(JsNativeError::typ() - .with_message("iterable next method not a function") - .into()); - }; + let next_method = self.next_method.as_callable().ok_or_else(|| { + JsNativeError::typ().with_message("iterable next method not a function") + })?; let result = if let Some(value) = value { // 2. Else, @@ -373,13 +369,14 @@ impl IteratorRecord { // 3. If Type(result) is not Object, throw a TypeError exception. // 4. Return result. - if let Some(o) = result.as_object() { - Ok(IteratorResult { object: o.clone() }) - } else { - Err(JsNativeError::typ() - .with_message("next value should be an object") - .into()) - } + result + .as_object() + .map(|o| IteratorResult { object: o.clone() }) + .ok_or_else(|| { + JsNativeError::typ() + .with_message("next value should be an object") + .into() + }) } /// `IteratorStep ( iteratorRecord )` diff --git a/boa_engine/src/builtins/map/mod.rs b/boa_engine/src/builtins/map/mod.rs index 9a128d1746..0713e75a2b 100644 --- a/boa_engine/src/builtins/map/mod.rs +++ b/boa_engine/src/builtins/map/mod.rs @@ -540,16 +540,14 @@ pub(crate) fn add_entries_from_iterable( // b. If next is false, return target. // c. Let nextItem be ? IteratorValue(next). - let next_item = if let Some(next) = next { - next.value(context)? - } else { + let Some(next_item) = next else { return Ok(target.clone().into()); }; - let next_item = if let Some(obj) = next_item.as_object() { - obj - // d. If Type(nextItem) is not Object, then - } else { + let next_item = next_item.value(context)?; + + let Some(next_item) = next_item.as_object() else { + // d. If Type(nextItem) is not Object, then // i. Let error be ThrowCompletion(a newly created TypeError object). let err = Err(JsNativeError::typ() .with_message("cannot get key and value from primitive item of `iterable`") diff --git a/boa_engine/src/builtins/object/mod.rs b/boa_engine/src/builtins/object/mod.rs index e64da8ac0f..a185a750d9 100644 --- a/boa_engine/src/builtins/object/mod.rs +++ b/boa_engine/src/builtins/object/mod.rs @@ -635,9 +635,7 @@ impl Object { } }; - let obj = if let Some(obj) = o.as_object() { - obj - } else { + let Some(obj) = o.as_object() else { // 3. If Type(O) is not Object, return O. return Ok(o); }; diff --git a/boa_engine/src/builtins/promise/mod.rs b/boa_engine/src/builtins/promise/mod.rs index ff0ee49acd..e127fff001 100644 --- a/boa_engine/src/builtins/promise/mod.rs +++ b/boa_engine/src/builtins/promise/mod.rs @@ -1268,23 +1268,21 @@ impl Promise { return Ok(JsValue::Undefined); } - let then = if let Some(resolution) = resolution.as_object() { - // 9. Let then be Completion(Get(resolution, "then")). - resolution.get("then", context) - } else { + let Some(then) = resolution.as_object() else { // 8. If Type(resolution) is not Object, then // a. Perform FulfillPromise(promise, resolution). promise - .borrow_mut() - .as_promise_mut() - .expect("Expected promise to be a Promise") - .fulfill_promise(resolution, context)?; + .borrow_mut() + .as_promise_mut() + .expect("Expected promise to be a Promise") + .fulfill_promise(resolution, context)?; // b. Return undefined. return Ok(JsValue::Undefined); }; - let then_action = match then { + // 9. Let then be Completion(Get(resolution, "then")). + let then_action = match then.get("then", context) { // 10. If then is an abrupt completion, then Err(e) => { // a. Perform RejectPromise(promise, then.[[Value]]). @@ -1741,9 +1739,7 @@ impl Promise { let promise = this; // 2. If Type(promise) is not Object, throw a TypeError exception. - let promise_obj = if let Some(p) = promise.as_object() { - p - } else { + let Some(promise_obj) = promise.as_object() else { return Err(JsNativeError::typ() .with_message("finally called with a non-object promise") .into()); diff --git a/boa_engine/src/builtins/reflect/mod.rs b/boa_engine/src/builtins/reflect/mod.rs index afa7ab2c8f..5544685100 100644 --- a/boa_engine/src/builtins/reflect/mod.rs +++ b/boa_engine/src/builtins/reflect/mod.rs @@ -113,13 +113,9 @@ impl Reflect { let new_target = if let Some(new_target) = args.get(2) { // 3. Else if IsConstructor(newTarget) is false, throw a TypeError exception. - if let Some(new_target) = new_target.as_constructor() { - new_target - } else { - return Err(JsNativeError::typ() - .with_message("newTarget must be a constructor") - .into()); - } + new_target.as_constructor().ok_or_else(|| { + JsNativeError::typ().with_message("newTarget must be a constructor") + })? } else { // 2. If newTarget is not present, set newTarget to target. target @@ -206,12 +202,11 @@ impl Reflect { // 2. Let key be ? ToPropertyKey(propertyKey). let key = args.get_or_undefined(1).to_property_key(context)?; // 3. If receiver is not present, then - let receiver = if let Some(receiver) = args.get(2).cloned() { - receiver - } else { - // 3.a. Set receiver to target. - target.clone().into() - }; + // 3.a. Set receiver to target. + let receiver = args + .get(2) + .cloned() + .unwrap_or_else(|| target.clone().into()); // 4. Return ? target.[[Get]](key, receiver). target.__get__(&key, receiver, context) } @@ -369,11 +364,10 @@ impl Reflect { .ok_or_else(|| JsNativeError::typ().with_message("target must be an object"))?; let key = args.get_or_undefined(1).to_property_key(context)?; let value = args.get_or_undefined(2); - let receiver = if let Some(receiver) = args.get(3).cloned() { - receiver - } else { - target.clone().into() - }; + let receiver = args + .get(3) + .cloned() + .unwrap_or_else(|| target.clone().into()); Ok(target .__set__(key, value.clone(), receiver, context)? .into()) diff --git a/boa_engine/src/builtins/regexp/mod.rs b/boa_engine/src/builtins/regexp/mod.rs index 632178bd4c..dd6f9f583d 100644 --- a/boa_engine/src/builtins/regexp/mod.rs +++ b/boa_engine/src/builtins/regexp/mod.rs @@ -1045,13 +1045,10 @@ impl RegExp { ) -> JsResult { // 1. Let rx be the this value. // 2. If Type(rx) is not Object, throw a TypeError exception. - let rx = if let Some(rx) = this.as_object() { - rx - } else { - return Err(JsNativeError::typ() + let rx = this.as_object().ok_or_else(|| { + JsNativeError::typ() .with_message("RegExp.prototype.match method called on incompatible value") - .into()); - }; + })?; // 3. Let S be ? ToString(string). let arg_str = args.get_or_undefined(0).to_string(context)?; @@ -1235,15 +1232,11 @@ impl RegExp { ) -> JsResult { // 1. Let rx be the this value. // 2. If Type(rx) is not Object, throw a TypeError exception. - let rx = if let Some(rx) = this.as_object() { - rx - } else { - return Err(JsNativeError::typ() - .with_message( - "RegExp.prototype[Symbol.replace] method called on incompatible value", - ) - .into()); - }; + let rx = this.as_object().ok_or_else(|| { + JsNativeError::typ().with_message( + "RegExp.prototype[Symbol.replace] method called on incompatible value", + ) + })?; // 3. Let S be ? ToString(string). let arg_str = args.get_or_undefined(0).to_string(context)?; @@ -1457,13 +1450,10 @@ impl RegExp { ) -> JsResult { // 1. Let rx be the this value. // 2. If Type(rx) is not Object, throw a TypeError exception. - let rx = if let Some(rx) = this.as_object() { - rx - } else { - return Err(JsNativeError::typ() + let rx = this.as_object().ok_or_else(|| { + JsNativeError::typ() .with_message("RegExp.prototype[Symbol.search] method called on incompatible value") - .into()); - }; + })?; // 3. Let S be ? ToString(string). let arg_str = args.get_or_undefined(0).to_string(context)?; @@ -1515,13 +1505,10 @@ impl RegExp { ) -> JsResult { // 1. Let rx be the this value. // 2. If Type(rx) is not Object, throw a TypeError exception. - let rx = if let Some(rx) = this.as_object() { - rx - } else { - return Err(JsNativeError::typ() + let rx = this.as_object().ok_or_else(|| { + JsNativeError::typ() .with_message("RegExp.prototype.split method called on incompatible value") - .into()); - }; + })?; // 3. Let S be ? ToString(string). let arg_str = args.get_or_undefined(0).to_string(context)?; diff --git a/boa_engine/src/builtins/set/mod.rs b/boa_engine/src/builtins/set/mod.rs index 7362701d74..6291fbcb9f 100644 --- a/boa_engine/src/builtins/set/mod.rs +++ b/boa_engine/src/builtins/set/mod.rs @@ -281,21 +281,16 @@ impl Set { pub(crate) fn delete(this: &JsValue, args: &[JsValue], _: &mut Context) -> JsResult { let value = args.get_or_undefined(0); - let res = if let Some(object) = this.as_object() { - if let Some(set) = object.borrow_mut().as_set_mut() { - set.delete(value) - } else { - return Err(JsNativeError::typ() - .with_message("'this' is not a Set") - .into()); - } - } else { - return Err(JsNativeError::typ() - .with_message("'this' is not a Set") - .into()); - }; + let mut object = this + .as_object() + .map(JsObject::borrow_mut) + .ok_or_else(|| JsNativeError::typ().with_message("'this' is not a Set"))?; + + let set = object + .as_set_mut() + .ok_or_else(|| JsNativeError::typ().with_message("'this' is not a Set"))?; - Ok(res.into()) + Ok(set.delete(value).into()) } /// `Set.prototype.entries( )` diff --git a/boa_engine/src/builtins/string/mod.rs b/boa_engine/src/builtins/string/mod.rs index a83a23f102..f9a5816c0d 100644 --- a/boa_engine/src/builtins/string/mod.rs +++ b/boa_engine/src/builtins/string/mod.rs @@ -952,9 +952,7 @@ impl String { // 8. Let position be ! StringIndexOf(string, searchString, 0). // 9. If position is -1, return string. - let position = if let Some(p) = this_str.index_of(&search_str, 0) { - p - } else { + let Some(position) = this_str.index_of(&search_str, 0) else { return Ok(this_str.into()); }; diff --git a/boa_engine/src/builtins/uri/mod.rs b/boa_engine/src/builtins/uri/mod.rs index 6278859891..123225579c 100644 --- a/boa_engine/src/builtins/uri/mod.rs +++ b/boa_engine/src/builtins/uri/mod.rs @@ -253,9 +253,7 @@ where let cp = string.code_point_at(k); // ii. If cp.[[IsUnpairedSurrogate]] is true, throw a URIError exception. - let ch = if let CodePoint::Unicode(ch) = cp { - ch - } else { + let CodePoint::Unicode(ch) = cp else { return Err(JsNativeError::uri() .with_message("trying to encode an invalid string") .into()); diff --git a/boa_engine/src/class.rs b/boa_engine/src/class.rs index d06c5b8530..69a5184a50 100644 --- a/boa_engine/src/class.rs +++ b/boa_engine/src/class.rs @@ -113,10 +113,8 @@ impl ClassConstructor for T { .into()); } - let class_constructor = context.global_object().clone().get(T::NAME, context)?; - let class_constructor = if let JsValue::Object(ref obj) = class_constructor { - obj - } else { + let class = context.global_object().clone().get(T::NAME, context)?; + let JsValue::Object(ref class_constructor) = class else { return Err(JsNativeError::typ() .with_message(format!( "invalid constructor for native class `{}` ", @@ -124,28 +122,25 @@ impl ClassConstructor for T { )) .into()); }; - let class_prototype = - if let JsValue::Object(ref obj) = class_constructor.get(PROTOTYPE, context)? { - obj.clone() - } else { - return Err(JsNativeError::typ() - .with_message(format!( - "invalid default prototype for native class `{}`", - T::NAME - )) - .into()); - }; + + let JsValue::Object(ref class_prototype) = class_constructor.get(PROTOTYPE, context)? else { + return Err(JsNativeError::typ() + .with_message(format!( + "invalid default prototype for native class `{}`", + T::NAME + )) + .into()); + }; let prototype = this .as_object() - .cloned() .map(|obj| { obj.get(PROTOTYPE, context) .map(|val| val.as_object().cloned()) }) .transpose()? .flatten() - .unwrap_or(class_prototype); + .unwrap_or_else(|| class_prototype.clone()); let native_instance = Self::constructor(this, args, context)?; let object_instance = JsObject::from_proto_and_data( diff --git a/boa_engine/src/lib.rs b/boa_engine/src/lib.rs index a43985f72c..791f12bb9f 100644 --- a/boa_engine/src/lib.rs +++ b/boa_engine/src/lib.rs @@ -72,7 +72,6 @@ // Ignore because `write!(string, ...)` instead of `string.push_str(&format!(...))` can fail. // We only use it in `ToInternedString` where performance is not an issue. clippy::format_push_string, - rustdoc::missing_doc_code_examples )] extern crate static_assertions as sa; diff --git a/boa_engine/src/object/internal_methods/arguments.rs b/boa_engine/src/object/internal_methods/arguments.rs index 7ade852d3f..23636d6624 100644 --- a/boa_engine/src/object/internal_methods/arguments.rs +++ b/boa_engine/src/object/internal_methods/arguments.rs @@ -30,9 +30,7 @@ pub(crate) fn arguments_exotic_get_own_property( ) -> JsResult> { // 1. Let desc be OrdinaryGetOwnProperty(args, P). // 2. If desc is undefined, return desc. - let desc = if let Some(desc) = super::ordinary_get_own_property(obj, key, context)? { - desc - } else { + let Some(desc) = super::ordinary_get_own_property(obj, key, context)? else { return Ok(None); }; diff --git a/boa_engine/src/object/internal_methods/global.rs b/boa_engine/src/object/internal_methods/global.rs index 5b5f8bdd2f..b8a9cf6b8a 100644 --- a/boa_engine/src/object/internal_methods/global.rs +++ b/boa_engine/src/object/internal_methods/global.rs @@ -501,11 +501,8 @@ pub(crate) fn validate_and_apply_property_descriptor( ); // 1. Assert: If O is not undefined, then IsPropertyKey(P) is true. - let mut current = if let Some(own) = current { - own - } - // 2. If current is undefined, then - else { + let Some(mut current) = current else { + // 2. If current is undefined, then // a. If extensible is false, return false. if !extensible { return false; diff --git a/boa_engine/src/object/internal_methods/mod.rs b/boa_engine/src/object/internal_methods/mod.rs index a8fc18514b..a2e8c88815 100644 --- a/boa_engine/src/object/internal_methods/mod.rs +++ b/boa_engine/src/object/internal_methods/mod.rs @@ -794,11 +794,8 @@ pub(crate) fn validate_and_apply_property_descriptor( Profiler::global().start_event("Object::validate_and_apply_property_descriptor", "object"); // 1. Assert: If O is not undefined, then IsPropertyKey(P) is true. - let mut current = if let Some(own) = current { - own - } - // 2. If current is undefined, then - else { + let Some(mut current) = current else { + // 2. If current is undefined, then // a. If extensible is false, return false. if !extensible { return false; diff --git a/boa_engine/src/object/internal_methods/proxy.rs b/boa_engine/src/object/internal_methods/proxy.rs index b50936c361..514e8b83b7 100644 --- a/boa_engine/src/object/internal_methods/proxy.rs +++ b/boa_engine/src/object/internal_methods/proxy.rs @@ -65,10 +65,8 @@ pub(crate) fn proxy_exotic_get_prototype_of( .try_data()?; // 5. Let trap be ? GetMethod(handler, "getPrototypeOf"). - let trap = if let Some(trap) = handler.get_method("getPrototypeOf", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("getPrototypeOf", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[GetPrototypeOf]](). return target.__get_prototype_of__(context); }; @@ -130,10 +128,8 @@ pub(crate) fn proxy_exotic_set_prototype_of( .try_data()?; // 5. Let trap be ? GetMethod(handler, "setPrototypeOf"). - let trap = if let Some(trap) = handler.get_method("setPrototypeOf", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("setPrototypeOf", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[SetPrototypeOf]](V). return target.__set_prototype_of__(val, context); }; @@ -193,10 +189,8 @@ pub(crate) fn proxy_exotic_is_extensible(obj: &JsObject, context: &mut Context) .try_data()?; // 5. Let trap be ? GetMethod(handler, "isExtensible"). - let trap = if let Some(trap) = handler.get_method("isExtensible", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("isExtensible", context)? else { + // 6. If trap is undefined, then // a. Return ? IsExtensible(target). return target.is_extensible(context); }; @@ -242,10 +236,8 @@ pub(crate) fn proxy_exotic_prevent_extensions( .try_data()?; // 5. Let trap be ? GetMethod(handler, "preventExtensions"). - let trap = if let Some(trap) = handler.get_method("preventExtensions", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("preventExtensions", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[PreventExtensions]](). return target.__prevent_extensions__(context); }; @@ -291,10 +283,8 @@ pub(crate) fn proxy_exotic_get_own_property( .try_data()?; // 5. Let trap be ? GetMethod(handler, "getOwnPropertyDescriptor"). - let trap = if let Some(trap) = handler.get_method("getOwnPropertyDescriptor", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("getOwnPropertyDescriptor", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[GetOwnProperty]](P). return target.__get_own_property__(key, context); }; @@ -418,10 +408,8 @@ pub(crate) fn proxy_exotic_define_own_property( .try_data()?; // 5. Let trap be ? GetMethod(handler, "defineProperty"). - let trap = if let Some(trap) = handler.get_method("defineProperty", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("defineProperty", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[DefineOwnProperty]](P, Desc). return target.__define_own_property__(key, desc, context); }; @@ -532,10 +520,8 @@ pub(crate) fn proxy_exotic_has_property( .try_data()?; // 5. Let trap be ? GetMethod(handler, "has"). - let trap = if let Some(trap) = handler.get_method("has", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("has", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[HasProperty]](P). return target.has_property(key.clone(), context); }; @@ -601,10 +587,8 @@ pub(crate) fn proxy_exotic_get( .try_data()?; // 5. Let trap be ? GetMethod(handler, "get"). - let trap = if let Some(trap) = handler.get_method("get", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("get", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[Get]](P, Receiver). return target.__get__(key, receiver, context); }; @@ -673,10 +657,8 @@ pub(crate) fn proxy_exotic_set( .try_data()?; // 5. Let trap be ? GetMethod(handler, "set"). - let trap = if let Some(trap) = handler.get_method("set", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("set", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[Set]](P, V, Receiver). return target.__set__(key, value, receiver, context); }; @@ -757,10 +739,8 @@ pub(crate) fn proxy_exotic_delete( .try_data()?; // 5. Let trap be ? GetMethod(handler, "deleteProperty"). - let trap = if let Some(trap) = handler.get_method("deleteProperty", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("deleteProperty", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[Delete]](P). return target.__delete__(key, context); }; @@ -826,10 +806,8 @@ pub(crate) fn proxy_exotic_own_property_keys( .try_data()?; // 5. Let trap be ? GetMethod(handler, "ownKeys"). - let trap = if let Some(trap) = handler.get_method("ownKeys", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("ownKeys", context)? else { + // 6. If trap is undefined, then // a. Return ? target.[[OwnPropertyKeys]](). return target.__own_property_keys__(context); }; @@ -964,10 +942,8 @@ fn proxy_exotic_call( .try_data()?; // 5. Let trap be ? GetMethod(handler, "apply"). - let trap = if let Some(trap) = handler.get_method("apply", context)? { - trap - // 6. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("apply", context)? else { + // 6. If trap is undefined, then // a. Return ? Call(target, thisArgument, argumentsList). return target.call(this, args, context); }; @@ -1009,10 +985,8 @@ fn proxy_exotic_construct( assert!(target.is_constructor()); // 6. Let trap be ? GetMethod(handler, "construct"). - let trap = if let Some(trap) = handler.get_method("construct", context)? { - trap - // 7. If trap is undefined, then - } else { + let Some(trap) = handler.get_method("construct", context)? else { + // 7. If trap is undefined, then // a. Return ? Construct(target, argumentsList, newTarget). return target.construct(args, Some(new_target), context); }; diff --git a/boa_engine/src/object/operations.rs b/boa_engine/src/object/operations.rs index 3a8c297605..8d340fda8c 100644 --- a/boa_engine/src/object/operations.rs +++ b/boa_engine/src/object/operations.rs @@ -502,13 +502,9 @@ impl JsObject { } // 4. If Type(C) is not Object, throw a TypeError exception. - let c = if let Some(c) = c.as_object() { - c - } else { - return Err(JsNativeError::typ() - .with_message("property 'constructor' is not an object") - .into()); - }; + let c = c.as_object().ok_or_else(|| { + JsNativeError::typ().with_message("property 'constructor' is not an object") + })?; // 5. Let S be ? Get(C, @@species). let s = c.get(WellKnownSymbols::species(), context)?; @@ -801,9 +797,7 @@ impl JsValue { context: &mut Context, ) -> JsResult { // 1. If IsCallable(C) is false, return false. - let function = if let Some(function) = function.as_callable() { - function - } else { + let Some(function) = function.as_callable() else { return Ok(false); }; @@ -818,9 +812,7 @@ impl JsValue { ); } - let mut object = if let Some(obj) = object.as_object() { - obj.clone() - } else { + let Some(mut object) = object.as_object().cloned() else { // 3. If Type(O) is not Object, return false. return Ok(false); }; @@ -828,14 +820,11 @@ impl JsValue { // 4. Let P be ? Get(C, "prototype"). let prototype = function.get("prototype", context)?; - let prototype = if let Some(obj) = prototype.as_object() { - obj - } else { - // 5. If Type(P) is not Object, throw a TypeError exception. - return Err(JsNativeError::typ() + // 5. If Type(P) is not Object, throw a TypeError exception. + let prototype = prototype.as_object().ok_or_else(|| { + JsNativeError::typ() .with_message("function has non-object prototype in instanceof check") - .into()); - }; + })?; // 6. Repeat, loop { diff --git a/boa_engine/src/string/mod.rs b/boa_engine/src/string/mod.rs index b8843b7cd2..dd0382b2ee 100644 --- a/boa_engine/src/string/mod.rs +++ b/boa_engine/src/string/mod.rs @@ -452,9 +452,7 @@ impl JsString { pub(crate) fn to_number(&self) -> f64 { // 1. Let text be ! StringToCodePoints(str). // 2. Let literal be ParseText(text, StringNumericLiteral). - let string = if let Ok(string) = self.to_std_string() { - string - } else { + let Ok(string) = self.to_std_string() else { // 3. If literal is a List of errors, return NaN. return f64::NAN; }; diff --git a/boa_engine/src/syntax/parser/expression/primary/object_initializer/mod.rs b/boa_engine/src/syntax/parser/expression/primary/object_initializer/mod.rs index 4e1fd01e3a..a26b1a19d4 100644 --- a/boa_engine/src/syntax/parser/expression/primary/object_initializer/mod.rs +++ b/boa_engine/src/syntax/parser/expression/primary/object_initializer/mod.rs @@ -216,11 +216,7 @@ where let (class_element_name, method) = AsyncMethod::new(self.allow_yield, self.allow_await).parse(cursor, interner)?; - let property_name = if let property::ClassElementName::PropertyName(property_name) = - class_element_name - { - property_name - } else { + let property::ClassElementName::PropertyName(property_name) = class_element_name else { return Err(ParseError::general( "private identifiers not allowed in object literal", position, diff --git a/boa_engine/src/syntax/parser/statement/declaration/hoistable/mod.rs b/boa_engine/src/syntax/parser/statement/declaration/hoistable/mod.rs index 6ca2cb8f2d..b6961ac306 100644 --- a/boa_engine/src/syntax/parser/statement/declaration/hoistable/mod.rs +++ b/boa_engine/src/syntax/parser/statement/declaration/hoistable/mod.rs @@ -143,24 +143,20 @@ fn parse_callable_declaration( cursor: &mut Cursor, interner: &mut Interner, ) -> Result<(Identifier, FormalParameterList, StatementList), ParseError> { - let next_token = cursor.peek(0, interner)?; - let name = if let Some(token) = next_token { - match token.kind() { - TokenKind::Punctuator(Punctuator::OpenParen) => { - if !c.is_default() { - return Err(ParseError::unexpected( - token.to_string(interner), - token.span(), - c.error_context(), - )); - } - Sym::DEFAULT.into() + let next_token = cursor.peek(0, interner)?.ok_or(ParseError::AbruptEnd)?; + let name = match next_token.kind() { + TokenKind::Punctuator(Punctuator::OpenParen) => { + if !c.is_default() { + return Err(ParseError::unexpected( + next_token.to_string(interner), + next_token.span(), + c.error_context(), + )); } - _ => BindingIdentifier::new(c.name_allow_yield(), c.name_allow_await()) - .parse(cursor, interner)?, + Sym::DEFAULT.into() } - } else { - return Err(ParseError::AbruptEnd); + _ => BindingIdentifier::new(c.name_allow_yield(), c.name_allow_await()) + .parse(cursor, interner)?, }; // Early Error: If BindingIdentifier is present and the source code matching BindingIdentifier is strict mode code, diff --git a/boa_engine/src/syntax/parser/statement/declaration/lexical.rs b/boa_engine/src/syntax/parser/statement/declaration/lexical.rs index 0f2bfca73f..feeb796949 100644 --- a/boa_engine/src/syntax/parser/statement/declaration/lexical.rs +++ b/boa_engine/src/syntax/parser/statement/declaration/lexical.rs @@ -261,20 +261,15 @@ where let bindings = ObjectBindingPattern::new(self.allow_yield, self.allow_await) .parse(cursor, interner)?; - let init = if let Some(t) = cursor.peek(0, interner)? { - if *t.kind() == TokenKind::Punctuator(Punctuator::Assign) { - Some( - Initializer::new( - None, - self.allow_in, - self.allow_yield, - self.allow_await, - ) + let init = if cursor + .peek(0, interner)? + .filter(|t| *t.kind() == TokenKind::Punctuator(Punctuator::Assign)) + .is_some() + { + Some( + Initializer::new(None, self.allow_in, self.allow_yield, self.allow_await) .parse(cursor, interner)?, - ) - } else { - None - } + ) } else { None }; @@ -294,20 +289,15 @@ where let bindings = ArrayBindingPattern::new(self.allow_yield, self.allow_await) .parse(cursor, interner)?; - let init = if let Some(t) = cursor.peek(0, interner)? { - if *t.kind() == TokenKind::Punctuator(Punctuator::Assign) { - Some( - Initializer::new( - None, - self.allow_in, - self.allow_yield, - self.allow_await, - ) + let init = if cursor + .peek(0, interner)? + .filter(|t| *t.kind() == TokenKind::Punctuator(Punctuator::Assign)) + .is_some() + { + Some( + Initializer::new(None, self.allow_in, self.allow_yield, self.allow_await) .parse(cursor, interner)?, - ) - } else { - None - } + ) } else { None }; @@ -334,20 +324,20 @@ where ))); } - let init = if let Some(t) = cursor.peek(0, interner)? { - if *t.kind() == TokenKind::Punctuator(Punctuator::Assign) { - Some( - Initializer::new( - Some(ident), - self.allow_in, - self.allow_yield, - self.allow_await, - ) - .parse(cursor, interner)?, + let init = if cursor + .peek(0, interner)? + .filter(|t| *t.kind() == TokenKind::Punctuator(Punctuator::Assign)) + .is_some() + { + Some( + Initializer::new( + Some(ident), + self.allow_in, + self.allow_yield, + self.allow_await, ) - } else { - None - } + .parse(cursor, interner)?, + ) } else { None }; diff --git a/boa_engine/src/syntax/parser/statement/variable/mod.rs b/boa_engine/src/syntax/parser/statement/variable/mod.rs index a243b106d8..6a41c34063 100644 --- a/boa_engine/src/syntax/parser/statement/variable/mod.rs +++ b/boa_engine/src/syntax/parser/statement/variable/mod.rs @@ -170,20 +170,15 @@ where let bindings = ObjectBindingPattern::new(self.allow_yield, self.allow_await) .parse(cursor, interner)?; - let init = if let Some(t) = cursor.peek(0, interner)? { - if *t.kind() == TokenKind::Punctuator(Punctuator::Assign) { - Some( - Initializer::new( - None, - self.allow_in, - self.allow_yield, - self.allow_await, - ) + let init = if cursor + .peek(0, interner)? + .filter(|t| *t.kind() == TokenKind::Punctuator(Punctuator::Assign)) + .is_some() + { + Some( + Initializer::new(None, self.allow_in, self.allow_yield, self.allow_await) .parse(cursor, interner)?, - ) - } else { - None - } + ) } else { None }; @@ -194,20 +189,15 @@ where let bindings = ArrayBindingPattern::new(self.allow_yield, self.allow_await) .parse(cursor, interner)?; - let init = if let Some(t) = cursor.peek(0, interner)? { - if *t.kind() == TokenKind::Punctuator(Punctuator::Assign) { - Some( - Initializer::new( - None, - self.allow_in, - self.allow_yield, - self.allow_await, - ) + let init = if cursor + .peek(0, interner)? + .filter(|t| *t.kind() == TokenKind::Punctuator(Punctuator::Assign)) + .is_some() + { + Some( + Initializer::new(None, self.allow_in, self.allow_yield, self.allow_await) .parse(cursor, interner)?, - ) - } else { - None - } + ) } else { None }; @@ -218,15 +208,15 @@ where let ident = BindingIdentifier::new(self.allow_yield, self.allow_await) .parse(cursor, interner)?; - let init = if let Some(t) = cursor.peek(0, interner)? { - if *t.kind() == TokenKind::Punctuator(Punctuator::Assign) { - Some( - Initializer::new(Some(ident), true, self.allow_yield, self.allow_await) - .parse(cursor, interner)?, - ) - } else { - None - } + let init = if cursor + .peek(0, interner)? + .filter(|t| *t.kind() == TokenKind::Punctuator(Punctuator::Assign)) + .is_some() + { + Some( + Initializer::new(Some(ident), true, self.allow_yield, self.allow_await) + .parse(cursor, interner)?, + ) } else { None }; diff --git a/boa_engine/src/vm/opcode/iteration/for_await.rs b/boa_engine/src/vm/opcode/iteration/for_await.rs index 0470cea3a8..5f71ffb3fb 100644 --- a/boa_engine/src/vm/opcode/iteration/for_await.rs +++ b/boa_engine/src/vm/opcode/iteration/for_await.rs @@ -23,13 +23,9 @@ impl Operation for ForAwaitOfLoopIterate { .as_boolean() .expect("iterator [[Done]] was not a boolean"); let next_method = context.vm.pop(); - let next_method_object = if let Some(object) = next_method.as_callable() { - object - } else { - return Err(JsNativeError::typ() - .with_message("iterable next method not a function") - .into()); - }; + let next_method_object = next_method.as_callable().ok_or_else(|| { + JsNativeError::typ().with_message("iterable next method not a function") + })?; let iterator = context.vm.pop(); let next_result = next_method_object.call(&iterator, &[], context)?; context.vm.push(iterator); @@ -54,13 +50,11 @@ impl Operation for ForAwaitOfLoopNext { let address = context.vm.read::(); let next_result = context.vm.pop(); - let next_result = if let Some(next_result) = next_result.as_object() { - IteratorResult::new(next_result.clone()) - } else { - return Err(JsNativeError::typ() - .with_message("next value should be an object") - .into()); - }; + let next_result = next_result + .as_object() + .cloned() + .map(IteratorResult::new) + .ok_or_else(|| JsNativeError::typ().with_message("next value should be an object"))?; if next_result.complete(context)? { context.vm.frame_mut().pc = address as usize; diff --git a/boa_interner/src/lib.rs b/boa_interner/src/lib.rs index 884dbc0bcc..112b3cd7e7 100644 --- a/boa_interner/src/lib.rs +++ b/boa_interner/src/lib.rs @@ -69,7 +69,6 @@ clippy::let_unit_value, // TODO deny once false positive is fixed (https://github.com/rust-lang/rust-clippy/issues/9626). clippy::trait_duplication_in_bounds, - rustdoc::missing_doc_code_examples, )] extern crate static_assertions as sa; diff --git a/boa_tester/src/main.rs b/boa_tester/src/main.rs index 2d2214af5e..54ef04d9a4 100644 --- a/boa_tester/src/main.rs +++ b/boa_tester/src/main.rs @@ -60,7 +60,6 @@ clippy::missing_errors_doc, clippy::as_conversions, clippy::let_unit_value, - rustdoc::missing_doc_code_examples )] mod exec; diff --git a/boa_unicode/src/lib.rs b/boa_unicode/src/lib.rs index ab673a803e..d896e788e0 100644 --- a/boa_unicode/src/lib.rs +++ b/boa_unicode/src/lib.rs @@ -63,8 +63,7 @@ clippy::must_use_candidate, clippy::missing_errors_doc, clippy::as_conversions, - clippy::let_unit_value, - rustdoc::missing_doc_code_examples + clippy::let_unit_value )] mod tables; diff --git a/boa_wasm/src/lib.rs b/boa_wasm/src/lib.rs index bfb70791e4..803ad9ecea 100644 --- a/boa_wasm/src/lib.rs +++ b/boa_wasm/src/lib.rs @@ -55,8 +55,7 @@ clippy::must_use_candidate, clippy::missing_errors_doc, clippy::as_conversions, - clippy::let_unit_value, - rustdoc::missing_doc_code_examples + clippy::let_unit_value )] use boa_engine::Context;