LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 17:58:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWasm: Make clang happy by removing an 'extra' set of parenthesis

Browse Source 
These aren't actually an extra set, without them the fold operation
would be syntactically invalid.
Also remove possible cast of float->double/double->float in Value::to()
This commit is contained in:
Ali Mohammad Pur 2021-05-03 23:59:20 +04:30 committed by Andreas Kling
parent 95b9821f26
commit 7fec66dd1c
Notes: sideshowbarker 2024-07-18 17:54:15 +09:00 
Author: https://github.com/alimpfard
Commit: https://github.com/SerenityOS/serenity/commit/7fec66dd1cb
Pull-request: https://github.com/SerenityOS/serenity/pull/7097
Reviewed-by: https://github.com/Dexesttp
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Userland/Libraries/LibWasm/AbstractMachine/AbstractMachine.h
View File

@ -102,7 +102,7 @@ public:
Optional<T> result;
m_value.visit(
[&](auto value) {
if constexpr (!IsSame<T, FunctionAddress> && !IsSame<T, ExternAddress>)
if constexpr (IsSame<T, decltype(value)>)
result = value;
},
[&](const FunctionAddress& address) {

4
Userland/Libraries/LibWasm/Parser/Parser.cpp
View File

@ -99,7 +99,9 @@ static ParseResult<ParseUntilAnyOfResult<T>> parse_until_any_of(InputStream& str
if (new_stream.has_any_error())
return with_eof_check(stream, ParseError::ExpectedValueOrTerminator);
if ((... || (byte == terminators))) {
constexpr auto equals = [](auto&& a, auto&& b) { return a == b; };
if ((... || equals(byte, terminators))) {
result.terminator = byte;
return result;
}