mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2025-04-22 12:35:14 +00:00
LibJS: Move join_args() in Interpreter
It can be useful outside of Runtime/ConsoleObject.cpp. join_args() => Interpreter::join_arguments()
This commit is contained in:
parent
046f9cf115
commit
73bead5ae9
Notes:
sideshowbarker
2024-07-19 06:59:02 +09:00
Author: https://github.com/emanuele6 Commit: https://github.com/SerenityOS/serenity/commit/73bead5ae91 Pull-request: https://github.com/SerenityOS/serenity/pull/2098 Reviewed-by: https://github.com/awesomekling
3 changed files with 20 additions and 17 deletions
|
@ -25,6 +25,7 @@
|
|||
*/
|
||||
|
||||
#include <AK/Badge.h>
|
||||
#include <AK/StringBuilder.h>
|
||||
#include <LibJS/AST.h>
|
||||
#include <LibJS/Interpreter.h>
|
||||
#include <LibJS/Runtime/Error.h>
|
||||
|
@ -261,4 +262,15 @@ const GlobalObject& Interpreter::global_object() const
|
|||
return static_cast<const GlobalObject&>(*m_global_object);
|
||||
}
|
||||
|
||||
String Interpreter::join_arguments()
|
||||
{
|
||||
StringBuilder joined_arguments;
|
||||
for (size_t i = 0; i < argument_count(); ++i) {
|
||||
joined_arguments.append(argument(i).to_string().characters());
|
||||
if (i != argument_count() - 1)
|
||||
joined_arguments.append(' ');
|
||||
}
|
||||
return joined_arguments.build();
|
||||
}
|
||||
|
||||
}
|
||||
|
|
|
@ -166,6 +166,8 @@ public:
|
|||
|
||||
Console& console() { return m_console; }
|
||||
|
||||
String join_arguments();
|
||||
|
||||
private:
|
||||
Interpreter();
|
||||
|
||||
|
|
|
@ -36,17 +36,6 @@
|
|||
|
||||
namespace JS {
|
||||
|
||||
static String join_args(Interpreter& interpreter)
|
||||
{
|
||||
StringBuilder joined_arguments;
|
||||
for (size_t i = 0; i < interpreter.argument_count(); ++i) {
|
||||
joined_arguments.append(interpreter.argument(i).to_string().characters());
|
||||
if (i != interpreter.argument_count() - 1)
|
||||
joined_arguments.append(' ');
|
||||
}
|
||||
return joined_arguments.build();
|
||||
}
|
||||
|
||||
ConsoleObject::ConsoleObject()
|
||||
: Object(interpreter().global_object().object_prototype())
|
||||
{
|
||||
|
@ -67,37 +56,37 @@ ConsoleObject::~ConsoleObject()
|
|||
|
||||
Value ConsoleObject::log(Interpreter& interpreter)
|
||||
{
|
||||
interpreter.console().log(join_args(interpreter));
|
||||
interpreter.console().log(interpreter.join_arguments());
|
||||
return js_undefined();
|
||||
}
|
||||
|
||||
Value ConsoleObject::debug(Interpreter& interpreter)
|
||||
{
|
||||
interpreter.console().debug(join_args(interpreter));
|
||||
interpreter.console().debug(interpreter.join_arguments());
|
||||
return js_undefined();
|
||||
}
|
||||
|
||||
Value ConsoleObject::info(Interpreter& interpreter)
|
||||
{
|
||||
interpreter.console().info(join_args(interpreter));
|
||||
interpreter.console().info(interpreter.join_arguments());
|
||||
return js_undefined();
|
||||
}
|
||||
|
||||
Value ConsoleObject::warn(Interpreter& interpreter)
|
||||
{
|
||||
interpreter.console().warn(join_args(interpreter));
|
||||
interpreter.console().warn(interpreter.join_arguments());
|
||||
return js_undefined();
|
||||
}
|
||||
|
||||
Value ConsoleObject::error(Interpreter& interpreter)
|
||||
{
|
||||
interpreter.console().error(join_args(interpreter));
|
||||
interpreter.console().error(interpreter.join_arguments());
|
||||
return js_undefined();
|
||||
}
|
||||
|
||||
Value ConsoleObject::trace(Interpreter& interpreter)
|
||||
{
|
||||
interpreter.console().trace(join_args(interpreter));
|
||||
interpreter.console().trace(interpreter.join_arguments());
|
||||
return js_undefined();
|
||||
}
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue