From 606a3982f34d777f121fcc4aa964141199676c20 Mon Sep 17 00:00:00 2001 From: Tim Schumacher Date: Thu, 9 Feb 2023 03:02:46 +0100 Subject: [PATCH] LibCore: Move Stream-based file into the `Core` namespace --- Ladybird/WebContent/main.cpp | 6 +- Ladybird/WebContentView.cpp | 2 +- Ladybird/main.cpp | 4 +- .../Tools/CodeGenerators/IPCCompiler/main.cpp | 5 +- .../CodeGenerators/LibEDID/GeneratePnpIDs.cpp | 15 +-- .../LibGL/GenerateGLAPIWrapper.cpp | 11 +- .../LibLocale/GenerateDateTimeFormatData.cpp | 8 +- .../LibLocale/GenerateLocaleData.cpp | 8 +- .../LibLocale/GenerateNumberFormatData.cpp | 8 +- .../LibLocale/GeneratePluralRulesData.cpp | 8 +- .../GenerateRelativeTimeFormatData.cpp | 8 +- .../LibTimeZone/GenerateTimeZoneData.cpp | 14 +-- .../LibUnicode/GenerateEmojiData.cpp | 20 ++-- .../LibUnicode/GenerateUnicodeData.cpp | 62 +++++----- .../CodeGenerators/LibUnicode/GeneratorUtil.h | 9 +- .../LibWeb/BindingsGenerator/main.cpp | 7 +- .../LibWeb/GenerateCSSEnums.cpp | 12 +- .../LibWeb/GenerateCSSMediaFeatureID.cpp | 12 +- .../LibWeb/GenerateCSSPropertyID.cpp | 12 +- .../LibWeb/GenerateCSSTransformFunctions.cpp | 12 +- .../LibWeb/GenerateCSSValueID.cpp | 12 +- .../GenerateWindowOrWorkerInterfaces.cpp | 11 +- .../CodeGenerators/LibWeb/GeneratorUtil.h | 3 +- .../StateMachineGenerator/main.cpp | 3 +- Meta/Lagom/Tools/ConfigureComponents/main.cpp | 2 +- ...Enable-java.lang.Process-on-serenity.patch | 5 +- Tests/LibCompress/TestBrotli.cpp | 7 +- Tests/LibCore/TestLibCoreStream.cpp | 27 ++--- Tests/LibCpp/test-cpp-parser.cpp | 3 +- Tests/LibCpp/test-cpp-preprocessor.cpp | 3 +- Tests/LibGL/TestRender.cpp | 3 +- Tests/LibJS/test-test262.cpp | 13 ++- Tests/LibJS/test262-runner.cpp | 9 +- Tests/LibMarkdown/TestCommonmark.cpp | 5 +- Tests/LibWasm/test-wasm.cpp | 2 +- Tests/LibWeb/TestHTMLTokenizer.cpp | 3 +- Userland/Applets/Network/main.cpp | 3 +- Userland/Applets/ResourceGraph/main.cpp | 10 +- .../Applications/Browser/BrowserWindow.cpp | 4 +- .../Applications/Browser/DownloadWidget.cpp | 2 +- .../Applications/Browser/DownloadWidget.h | 2 +- Userland/Applications/Browser/main.cpp | 4 +- .../ContentFilterSettingsWidget.cpp | 6 +- .../DisplaySettings/ThemePreviewWidget.cpp | 2 +- .../FileOperationProgressWidget.cpp | 2 +- .../FileManager/FileOperationProgressWidget.h | 5 +- .../Applications/FileManager/FileUtils.cpp | 4 +- .../Applications/HexEditor/HexDocument.cpp | 13 ++- Userland/Applications/HexEditor/HexDocument.h | 14 +-- Userland/Applications/HexEditor/HexEditor.cpp | 4 +- Userland/Applications/HexEditor/HexEditor.h | 4 +- .../HexEditor/HexEditorWidget.cpp | 8 +- .../Applications/HexEditor/HexEditorWidget.h | 2 +- Userland/Applications/HexEditor/main.cpp | 2 +- .../KeyboardMapper/KeyboardMapperWidget.cpp | 3 +- .../NetworkSettings/NetworkSettingsWidget.cpp | 2 +- .../PDFViewer/PDFViewerWidget.cpp | 4 +- .../Applications/PDFViewer/PDFViewerWidget.h | 4 +- .../Applications/PixelPaint/ImageEditor.cpp | 4 +- .../Applications/PixelPaint/ImageEditor.h | 2 +- .../Applications/PixelPaint/MainWidget.cpp | 2 +- .../Applications/PixelPaint/PaletteWidget.cpp | 8 +- .../Applications/PixelPaint/PaletteWidget.h | 4 +- .../Applications/PixelPaint/ProjectLoader.cpp | 2 +- .../Applications/PixelPaint/ProjectLoader.h | 2 +- .../Applications/Presenter/Presentation.cpp | 3 +- Userland/Applications/Run/RunWindow.cpp | 6 +- .../Applications/SoundPlayer/M3UParser.cpp | 3 +- .../SpaceAnalyzer/TreeMapWidget.cpp | 2 +- .../Applications/Spreadsheet/ExportDialog.cpp | 2 +- .../Applications/Spreadsheet/ExportDialog.h | 2 +- .../Applications/Spreadsheet/ImportDialog.cpp | 2 +- .../Applications/Spreadsheet/ImportDialog.h | 2 +- .../Spreadsheet/SpreadsheetWidget.cpp | 8 +- .../Spreadsheet/SpreadsheetWidget.h | 6 +- .../Applications/Spreadsheet/Workbook.cpp | 6 +- Userland/Applications/Spreadsheet/Workbook.h | 6 +- .../SystemMonitor/ProcessModel.cpp | 2 +- .../Applications/TextEditor/MainWidget.cpp | 4 +- Userland/Applications/TextEditor/MainWidget.h | 2 +- .../Applications/ThemeEditor/MainWidget.cpp | 12 +- .../Applications/ThemeEditor/MainWidget.h | 4 +- .../Applications/Welcome/WelcomeWidget.cpp | 4 +- Userland/Demos/CatDog/CatDog.cpp | 3 +- Userland/Demos/CatDog/CatDog.h | 2 +- Userland/DevTools/GMLPlayground/main.cpp | 2 +- Userland/DevTools/HackStudio/Editor.cpp | 2 +- .../DevTools/HackStudio/Git/GitWidget.cpp | 2 +- .../DevTools/HackStudio/HackStudioWidget.cpp | 4 +- .../HackStudio/LanguageServers/FileDB.cpp | 7 +- .../HackStudio/LanguageServers/FileDB.h | 2 +- .../DevTools/HackStudio/ProjectBuilder.cpp | 4 +- .../DevTools/HackStudio/ProjectConfig.cpp | 3 +- Userland/DevTools/HackStudio/ProjectFile.cpp | 3 +- Userland/DevTools/Profiler/Profile.cpp | 2 +- Userland/DevTools/Profiler/SourceModel.cpp | 4 +- Userland/DevTools/SQLStudio/ScriptEditor.cpp | 4 +- Userland/DevTools/UserspaceEmulator/main.cpp | 2 +- Userland/Games/Chess/ChessWidget.cpp | 5 +- Userland/Games/Chess/ChessWidget.h | 4 +- Userland/Games/MasterWord/WordGame.cpp | 4 +- Userland/Libraries/LibAudio/FlacLoader.cpp | 3 +- Userland/Libraries/LibAudio/MP3Loader.cpp | 3 +- Userland/Libraries/LibAudio/WavLoader.cpp | 3 +- Userland/Libraries/LibCore/CMakeLists.txt | 2 +- Userland/Libraries/LibCore/ConfigFile.cpp | 14 +-- Userland/Libraries/LibCore/ConfigFile.h | 7 +- Userland/Libraries/LibCore/DeprecatedFile.h | 2 +- Userland/Libraries/LibCore/Directory.cpp | 6 +- Userland/Libraries/LibCore/Directory.h | 3 +- .../LibCore/{Stream.cpp => File.cpp} | 16 +-- Userland/Libraries/LibCore/File.h | 108 ++++++++++++++++++ Userland/Libraries/LibCore/Forward.h | 5 +- .../LibCore/ProcessStatisticsReader.cpp | 3 +- Userland/Libraries/LibCore/Stream.h | 95 --------------- .../LibFileSystemAccessClient/Client.cpp | 26 ++--- .../LibFileSystemAccessClient/Client.h | 14 +-- .../Libraries/LibGUI/AbstractThemePreview.cpp | 2 +- .../Libraries/LibGUI/AbstractThemePreview.h | 2 +- .../LibGUI/CommonLocationsProvider.cpp | 3 +- Userland/Libraries/LibGUI/TextEditor.cpp | 4 +- Userland/Libraries/LibGUI/TextEditor.h | 2 +- Userland/Libraries/LibGfx/Font/BitmapFont.cpp | 3 +- Userland/Libraries/LibIDL/IDLParser.cpp | 3 +- Userland/Libraries/LibIPC/File.h | 3 +- Userland/Libraries/LibLine/Editor.cpp | 14 +-- .../Libraries/LibLine/InternalFunctions.cpp | 9 +- .../LibLine/XtermSuggestionDisplay.cpp | 5 +- Userland/Libraries/LibProtocol/Request.cpp | 2 +- Userland/Libraries/LibSQL/Heap.cpp | 4 +- Userland/Libraries/LibSQL/Heap.h | 3 +- Userland/Libraries/LibSQL/SQLClient.cpp | 4 +- Userland/Libraries/LibTLS/HandshakeClient.cpp | 2 +- .../Libraries/LibTest/JavaScriptTestRunner.h | 3 +- Userland/Libraries/LibVirtGPU/Device.cpp | 5 +- Userland/Libraries/LibVirtGPU/Device.h | 4 +- .../LibWeb/Loader/ResourceLoader.cpp | 2 +- Userland/Services/FileOperation/main.cpp | 9 +- .../ConnectionFromClient.cpp | 36 +++--- .../ConnectionFromClient.h | 12 +- .../FileSystemAccessServer.ipc | 6 +- Userland/Services/NetworkServer/main.cpp | 2 +- .../Services/RequestServer/GeminiProtocol.cpp | 2 +- .../Services/RequestServer/GeminiRequest.cpp | 4 +- .../Services/RequestServer/GeminiRequest.h | 4 +- Userland/Services/RequestServer/HttpCommon.h | 2 +- .../Services/RequestServer/HttpRequest.cpp | 4 +- Userland/Services/RequestServer/HttpRequest.h | 4 +- .../Services/RequestServer/HttpsRequest.cpp | 4 +- .../Services/RequestServer/HttpsRequest.h | 4 +- Userland/Services/RequestServer/Request.cpp | 2 +- Userland/Services/RequestServer/Request.h | 6 +- Userland/Services/WebServer/Client.cpp | 3 +- Userland/Utilities/arp.cpp | 3 +- Userland/Utilities/base64.cpp | 3 +- Userland/Utilities/cat.cpp | 5 +- Userland/Utilities/checksum.cpp | 5 +- Userland/Utilities/cksum.cpp | 3 +- Userland/Utilities/cmp.cpp | 11 +- Userland/Utilities/comm.cpp | 9 +- Userland/Utilities/copy.cpp | 3 +- Userland/Utilities/cpp-lexer.cpp | 3 +- Userland/Utilities/cpp-parser.cpp | 3 +- Userland/Utilities/cpp-preprocessor.cpp | 3 +- Userland/Utilities/df.cpp | 3 +- Userland/Utilities/diff.cpp | 5 +- Userland/Utilities/dmesg.cpp | 3 +- Userland/Utilities/du.cpp | 3 +- Userland/Utilities/file.cpp | 3 +- Userland/Utilities/fortune.cpp | 3 +- Userland/Utilities/gml-format.cpp | 5 +- Userland/Utilities/grep.cpp | 5 +- Userland/Utilities/gron.cpp | 3 +- Userland/Utilities/gunzip.cpp | 7 +- Userland/Utilities/gzip.cpp | 3 +- Userland/Utilities/headless-browser.cpp | 2 +- Userland/Utilities/hexdump.cpp | 3 +- Userland/Utilities/ifconfig.cpp | 3 +- Userland/Utilities/js.cpp | 10 +- Userland/Utilities/json.cpp | 3 +- Userland/Utilities/lsblk.cpp | 7 +- Userland/Utilities/lscpu.cpp | 3 +- Userland/Utilities/lsirq.cpp | 3 +- Userland/Utilities/lsjails.cpp | 3 +- Userland/Utilities/lsof.cpp | 3 +- Userland/Utilities/lspci.cpp | 13 ++- Userland/Utilities/lsusb.cpp | 3 +- Userland/Utilities/man.cpp | 3 +- Userland/Utilities/markdown-check.cpp | 3 +- Userland/Utilities/md.cpp | 3 +- Userland/Utilities/mount.cpp | 7 +- Userland/Utilities/netstat.cpp | 5 +- Userland/Utilities/nologin.cpp | 3 +- Userland/Utilities/nproc.cpp | 3 +- Userland/Utilities/pmap.cpp | 3 +- Userland/Utilities/pro.cpp | 2 +- Userland/Utilities/reboot.cpp | 3 +- Userland/Utilities/route.cpp | 3 +- Userland/Utilities/sed.cpp | 9 +- Userland/Utilities/shot.cpp | 2 +- Userland/Utilities/shuf.cpp | 3 +- Userland/Utilities/shutdown.cpp | 3 +- Userland/Utilities/sort.cpp | 5 +- Userland/Utilities/sql.cpp | 6 +- Userland/Utilities/strace.cpp | 5 +- Userland/Utilities/strings.cpp | 3 +- Userland/Utilities/sysctl.cpp | 5 +- Userland/Utilities/tail.cpp | 7 +- Userland/Utilities/tar.cpp | 6 +- Userland/Utilities/test-fuzz.cpp | 3 +- Userland/Utilities/tsort.cpp | 3 +- Userland/Utilities/uniq.cpp | 7 +- Userland/Utilities/uptime.cpp | 3 +- Userland/Utilities/utmpupdate.cpp | 3 +- Userland/Utilities/w.cpp | 3 +- Userland/Utilities/wasm.cpp | 3 +- Userland/Utilities/xml.cpp | 9 +- Userland/Utilities/zip.cpp | 5 +- 218 files changed, 748 insertions(+), 643 deletions(-) rename Userland/Libraries/LibCore/{Stream.cpp => File.cpp} (93%) create mode 100644 Userland/Libraries/LibCore/File.h diff --git a/Ladybird/WebContent/main.cpp b/Ladybird/WebContent/main.cpp index 9cce282d0837ae..d1ab62ad9b5222 100644 --- a/Ladybird/WebContent/main.cpp +++ b/Ladybird/WebContent/main.cpp @@ -111,13 +111,13 @@ ErrorOr serenity_main(Main::Arguments arguments) static ErrorOr load_content_filters() { - auto file_or_error = Core::Stream::File::open(DeprecatedString::formatted("{}/home/anon/.config/BrowserContentFilters.txt", s_serenity_resource_root), Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(DeprecatedString::formatted("{}/home/anon/.config/BrowserContentFilters.txt", s_serenity_resource_root), Core::File::OpenMode::Read); if (file_or_error.is_error()) - file_or_error = Core::Stream::File::open(DeprecatedString::formatted("{}/res/ladybird/BrowserContentFilters.txt", s_serenity_resource_root), Core::Stream::OpenMode::Read); + file_or_error = Core::File::open(DeprecatedString::formatted("{}/res/ladybird/BrowserContentFilters.txt", s_serenity_resource_root), Core::File::OpenMode::Read); if (file_or_error.is_error()) return file_or_error.release_error(); auto file = file_or_error.release_value(); - auto ad_filter_list = TRY(Core::Stream::BufferedFile::create(move(file))); + auto ad_filter_list = TRY(Core::BufferedFile::create(move(file))); auto buffer = TRY(ByteBuffer::create_uninitialized(4096)); while (TRY(ad_filter_list->can_read_line())) { auto line = TRY(ad_filter_list->read_line(buffer)); diff --git a/Ladybird/WebContentView.cpp b/Ladybird/WebContentView.cpp index e35b7c8310a9e3..abc3d74763b09e 100644 --- a/Ladybird/WebContentView.cpp +++ b/Ladybird/WebContentView.cpp @@ -1006,7 +1006,7 @@ Gfx::IntRect WebContentView::notify_server_did_request_fullscreen_window() void WebContentView::notify_server_did_request_file(Badge, DeprecatedString const& path, i32 request_id) { - auto file = Core::Stream::File::open(path, Core::Stream::OpenMode::Read); + auto file = Core::File::open(path, Core::File::OpenMode::Read); if (file.is_error()) client().async_handle_file_return(file.error().code(), {}, request_id); else diff --git a/Ladybird/main.cpp b/Ladybird/main.cpp index f29490a05018db..497a7b4929c041 100644 --- a/Ladybird/main.cpp +++ b/Ladybird/main.cpp @@ -31,8 +31,8 @@ static ErrorOr handle_attached_debugger() // incorrectly forwards the signal to us even when it's set to // "nopass". See https://sourceware.org/bugzilla/show_bug.cgi?id=9425 // for details. - auto unbuffered_status_file = TRY(Core::Stream::File::open("/proc/self/status"sv, Core::Stream::OpenMode::Read)); - auto status_file = TRY(Core::Stream::BufferedFile::create(move(unbuffered_status_file))); + auto unbuffered_status_file = TRY(Core::File::open("/proc/self/status"sv, Core::File::OpenMode::Read)); + auto status_file = TRY(Core::BufferedFile::create(move(unbuffered_status_file))); auto buffer = TRY(ByteBuffer::create_uninitialized(4096)); while (TRY(status_file->can_read_line())) { auto line = TRY(status_file->read_line(buffer)); diff --git a/Meta/Lagom/Tools/CodeGenerators/IPCCompiler/main.cpp b/Meta/Lagom/Tools/CodeGenerators/IPCCompiler/main.cpp index 2a106802a3556f..6f7b1dfee62c64 100644 --- a/Meta/Lagom/Tools/CodeGenerators/IPCCompiler/main.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/IPCCompiler/main.cpp @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -69,7 +70,7 @@ static bool is_primitive_type(DeprecatedString const& type) static bool is_simple_type(DeprecatedString const& type) { // Small types that it makes sense just to pass by value. - return type.is_one_of("Gfx::Color", "Gfx::IntPoint", "Gfx::FloatPoint", "Gfx::IntSize", "Gfx::FloatSize", "Core::Stream::OpenMode"); + return type.is_one_of("Gfx::Color", "Gfx::IntPoint", "Gfx::FloatPoint", "Gfx::IntSize", "Gfx::FloatSize", "Core::File::OpenMode"); } static bool is_primitive_or_simple_type(DeprecatedString const& type) @@ -796,7 +797,7 @@ ErrorOr serenity_main(Main::Arguments arguments) return 1; } - auto file = TRY(Core::Stream::File::open(arguments.strings[1], Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(arguments.strings[1], Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibEDID/GeneratePnpIDs.cpp b/Meta/Lagom/Tools/CodeGenerators/LibEDID/GeneratePnpIDs.cpp index ebd8ad8623b2b7..fca6c8d1cbc348 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibEDID/GeneratePnpIDs.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibEDID/GeneratePnpIDs.cpp @@ -6,6 +6,7 @@ #include #include +#include #include enum class PnpIdColumns { @@ -116,7 +117,7 @@ static ErrorOr parse_approval_date(StringView const& str) return ApprovalDate { .year = year.value(), .month = month.value(), .day = day.value() }; } -static ErrorOr> parse_pnp_ids_database(Core::Stream::File& pnp_ids_file) +static ErrorOr> parse_pnp_ids_database(Core::File& pnp_ids_file) { auto pnp_ids_file_bytes = TRY(pnp_ids_file.read_until_eof()); StringView pnp_ids_file_contents(pnp_ids_file_bytes); @@ -181,7 +182,7 @@ static ErrorOr> parse_pnp_ids_database(Core return pnp_id_data; } -static ErrorOr generate_header(Core::Stream::File& file, HashMap const& pnp_ids) +static ErrorOr generate_header(Core::File& file, HashMap const& pnp_ids) { StringBuilder builder; SourceGenerator generator { builder }; @@ -215,7 +216,7 @@ namespace PnpIDs { return {}; } -static ErrorOr generate_source(Core::Stream::File& file, HashMap const& pnp_ids) +static ErrorOr generate_source(Core::File& file, HashMap const& pnp_ids) { StringBuilder builder; SourceGenerator generator { builder }; @@ -281,17 +282,17 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(pnp_ids_file_path, "Path to the input PNP ID database file", "pnp-ids-file", 'p', "pnp-ids-file"); args_parser.parse(arguments); - auto open_file = [&](StringView path, Core::Stream::OpenMode mode = Core::Stream::OpenMode::Read) -> ErrorOr> { + auto open_file = [&](StringView path, Core::File::OpenMode mode = Core::File::OpenMode::Read) -> ErrorOr> { if (path.is_empty()) { args_parser.print_usage(stderr, arguments.argv[0]); return Error::from_string_literal("Must provide all command line options"); } - return Core::Stream::File::open(path, mode); + return Core::File::open(path, mode); }; - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::ReadWrite)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::ReadWrite)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::ReadWrite)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::ReadWrite)); auto pnp_ids_file = TRY(open_file(pnp_ids_file_path)); auto pnp_id_map = TRY(parse_pnp_ids_database(*pnp_ids_file)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibGL/GenerateGLAPIWrapper.cpp b/Meta/Lagom/Tools/CodeGenerators/LibGL/GenerateGLAPIWrapper.cpp index e44191c93d0cf7..99bf32fe44bbd2 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibGL/GenerateGLAPIWrapper.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibGL/GenerateGLAPIWrapper.cpp @@ -14,6 +14,7 @@ #include #include #include +#include #include #include @@ -355,7 +356,7 @@ Vector create_function_definitions(DeprecatedString function return functions; } -ErrorOr generate_header_file(JsonObject& api_data, Core::Stream::File& file) +ErrorOr generate_header_file(JsonObject& api_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -409,7 +410,7 @@ ErrorOr generate_header_file(JsonObject& api_data, Core::Stream::File& fil return {}; } -ErrorOr generate_implementation_file(JsonObject& api_data, Core::Stream::File& file) +ErrorOr generate_implementation_file(JsonObject& api_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -535,7 +536,7 @@ ErrorOr generate_implementation_file(JsonObject& api_data, Core::Stream::F ErrorOr read_entire_file_as_json(StringView filename) { - auto file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(filename, Core::File::OpenMode::Read)); auto json_size = TRY(file->size()); auto json_data = TRY(ByteBuffer::create_uninitialized(json_size)); TRY(file->read_entire_buffer(json_data.bytes())); @@ -558,8 +559,8 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(json.is_object()); auto api_data = json.as_object(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_header_file(api_data, *generated_header_file)); TRY(generate_implementation_file(api_data, *generated_implementation_file)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateDateTimeFormatData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateDateTimeFormatData.cpp index c54b4fbd45d409..b33a4a17c2d6a4 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateDateTimeFormatData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateDateTimeFormatData.cpp @@ -1701,7 +1701,7 @@ static DeprecatedString format_identifier(StringView owner, DeprecatedString ide return identifier; } -static ErrorOr generate_unicode_locale_header(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_header(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -1729,7 +1729,7 @@ namespace Locale { return {}; } -static ErrorOr generate_unicode_locale_implementation(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_implementation(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -2416,8 +2416,8 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(dates_path, "Path to cldr-dates directory", "dates-path", 'd', "dates-path"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); CLDR cldr; TRY(parse_all_locales(core_path, dates_path, cldr)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateLocaleData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateLocaleData.cpp index f2e0249bcfbb35..3a2cd6ebdaf3ec 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateLocaleData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateLocaleData.cpp @@ -1007,7 +1007,7 @@ static ErrorOr parse_all_locales(DeprecatedString bcp47_path, DeprecatedSt return {}; } -static ErrorOr generate_unicode_locale_header(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_header(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -1052,7 +1052,7 @@ namespace Locale { return {}; } -static ErrorOr generate_unicode_locale_implementation(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_implementation(Core::BufferedFile& file, CLDR& cldr) { auto string_index_type = cldr.unique_strings.type_that_fits(); @@ -1774,8 +1774,8 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(dates_path, "Path to cldr-dates directory", "dates-path", 'd', "dates-path"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); CLDR cldr; TRY(parse_all_locales(bcp47_path, core_path, locale_names_path, misc_path, numbers_path, dates_path, cldr)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateNumberFormatData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateNumberFormatData.cpp index f9c35fae1b455e..7609d9e7b79cc2 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateNumberFormatData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateNumberFormatData.cpp @@ -745,7 +745,7 @@ static DeprecatedString format_identifier(StringView, DeprecatedString identifie return identifier.to_titlecase(); } -static ErrorOr generate_unicode_locale_header(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_header(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -768,7 +768,7 @@ namespace Locale { return {}; } -static ErrorOr generate_unicode_locale_implementation(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_implementation(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -1128,8 +1128,8 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(units_path, "Path to cldr-units directory", "units-path", 'u', "units-path"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); CLDR cldr; TRY(parse_all_locales(core_path, numbers_path, units_path, cldr)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GeneratePluralRulesData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GeneratePluralRulesData.cpp index d13b5f375334ec..70ce71efe0732e 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GeneratePluralRulesData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GeneratePluralRulesData.cpp @@ -426,7 +426,7 @@ static ErrorOr parse_all_locales(DeprecatedString core_path, DeprecatedStr return {}; } -static ErrorOr generate_unicode_locale_header(Core::Stream::BufferedFile& file, CLDR&) +static ErrorOr generate_unicode_locale_header(Core::BufferedFile& file, CLDR&) { StringBuilder builder; SourceGenerator generator { builder }; @@ -447,7 +447,7 @@ namespace Locale { return {}; } -static ErrorOr generate_unicode_locale_implementation(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_implementation(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -673,8 +673,8 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(locale_names_path, "Path to cldr-localenames directory", "locale-names-path", 'l', "locale-names-path"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); CLDR cldr; TRY(parse_all_locales(core_path, locale_names_path, cldr)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateRelativeTimeFormatData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateRelativeTimeFormatData.cpp index 0ec821c8a67a8c..382c6b84725aa2 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateRelativeTimeFormatData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibLocale/GenerateRelativeTimeFormatData.cpp @@ -164,7 +164,7 @@ static ErrorOr parse_all_locales(DeprecatedString dates_path, CLDR& cldr) return {}; } -static ErrorOr generate_unicode_locale_header(Core::Stream::BufferedFile& file, CLDR&) +static ErrorOr generate_unicode_locale_header(Core::BufferedFile& file, CLDR&) { StringBuilder builder; SourceGenerator generator { builder }; @@ -185,7 +185,7 @@ namespace Locale { return {}; } -static ErrorOr generate_unicode_locale_implementation(Core::Stream::BufferedFile& file, CLDR& cldr) +static ErrorOr generate_unicode_locale_implementation(Core::BufferedFile& file, CLDR& cldr) { StringBuilder builder; SourceGenerator generator { builder }; @@ -293,8 +293,8 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(dates_path, "Path to cldr-dates directory", "dates-path", 'd', "dates-path"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); CLDR cldr; TRY(parse_all_locales(dates_path, cldr)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibTimeZone/GenerateTimeZoneData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibTimeZone/GenerateTimeZoneData.cpp index a1839d01de39f6..0290e46a7768ea 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibTimeZone/GenerateTimeZoneData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibTimeZone/GenerateTimeZoneData.cpp @@ -317,7 +317,7 @@ static void parse_rule(StringView rule_line, TimeZoneData& time_zone_data) static ErrorOr parse_time_zones(StringView time_zone_path, TimeZoneData& time_zone_data) { // For reference, the man page for `zic` has the best documentation of the TZDB file format. - auto file = TRY(open_file(time_zone_path, Core::Stream::OpenMode::Read)); + auto file = TRY(open_file(time_zone_path, Core::File::OpenMode::Read)); Array buffer {}; Vector* last_parsed_zone = nullptr; @@ -346,7 +346,7 @@ static ErrorOr parse_time_zones(StringView time_zone_path, TimeZoneData& t return {}; } -static ErrorOr parse_time_zone_coordinates(Core::Stream::BufferedFile& file, TimeZoneData& time_zone_data) +static ErrorOr parse_time_zone_coordinates(Core::BufferedFile& file, TimeZoneData& time_zone_data) { auto parse_coordinate = [](auto coordinate) { VERIFY(coordinate.substring_view(0, 1).is_one_of("+"sv, "-"sv)); @@ -448,7 +448,7 @@ static DeprecatedString format_identifier(StringView owner, DeprecatedString ide return identifier; } -static ErrorOr generate_time_zone_data_header(Core::Stream::BufferedFile& file, TimeZoneData& time_zone_data) +static ErrorOr generate_time_zone_data_header(Core::BufferedFile& file, TimeZoneData& time_zone_data) { StringBuilder builder; SourceGenerator generator { builder }; @@ -473,7 +473,7 @@ namespace TimeZone { return {}; } -static ErrorOr generate_time_zone_data_implementation(Core::Stream::BufferedFile& file, TimeZoneData& time_zone_data) +static ErrorOr generate_time_zone_data_implementation(Core::BufferedFile& file, TimeZoneData& time_zone_data) { StringBuilder builder; SourceGenerator generator { builder }; @@ -819,9 +819,9 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(time_zone_paths, "Paths to the time zone database files", "time-zone-paths"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); - auto time_zone_coordinates_file = TRY(open_file(time_zone_coordinates_path, Core::Stream::OpenMode::Read)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); + auto time_zone_coordinates_file = TRY(open_file(time_zone_coordinates_path, Core::File::OpenMode::Read)); TimeZoneData time_zone_data {}; for (auto time_zone_path : time_zone_paths) diff --git a/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateEmojiData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateEmojiData.cpp index 984f44396c0154..ca792f4167fad9 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateEmojiData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateEmojiData.cpp @@ -50,7 +50,7 @@ static void set_image_path_for_emoji(StringView emoji_resource_path, Emoji& emoj emoji.image_path = move(path); } -static ErrorOr parse_emoji_test_data(Core::Stream::BufferedFile& file, EmojiData& emoji_data) +static ErrorOr parse_emoji_test_data(Core::BufferedFile& file, EmojiData& emoji_data) { static constexpr auto group_header = "# group: "sv; static constexpr auto subgroup_header = "# subgroup: "sv; @@ -114,7 +114,7 @@ static ErrorOr parse_emoji_test_data(Core::Stream::BufferedFile& file, Emo return {}; } -static ErrorOr parse_emoji_serenity_data(Core::Stream::BufferedFile& file, EmojiData& emoji_data) +static ErrorOr parse_emoji_serenity_data(Core::BufferedFile& file, EmojiData& emoji_data) { static constexpr auto code_point_header = "U+"sv; @@ -164,7 +164,7 @@ static ErrorOr parse_emoji_serenity_data(Core::Stream::BufferedFile& file, return {}; } -static ErrorOr generate_emoji_data_header(Core::Stream::BufferedFile& file, EmojiData const&) +static ErrorOr generate_emoji_data_header(Core::BufferedFile& file, EmojiData const&) { StringBuilder builder; SourceGenerator generator { builder }; @@ -173,7 +173,7 @@ static ErrorOr generate_emoji_data_header(Core::Stream::BufferedFile& file return {}; } -static ErrorOr generate_emoji_data_implementation(Core::Stream::BufferedFile& file, EmojiData const& emoji_data) +static ErrorOr generate_emoji_data_implementation(Core::BufferedFile& file, EmojiData const& emoji_data) { StringBuilder builder; SourceGenerator generator { builder }; @@ -276,7 +276,7 @@ Optional find_emoji_for_code_points(ReadonlySpan code_points) return {}; } -static ErrorOr generate_emoji_installation(Core::Stream::BufferedFile& file, EmojiData const& emoji_data) +static ErrorOr generate_emoji_installation(Core::BufferedFile& file, EmojiData const& emoji_data) { StringBuilder builder; SourceGenerator generator { builder }; @@ -339,14 +339,14 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(emoji_resource_path, "Path to the /res/emoji directory", "emoji-resource-path", 'r', "emoji-resource-path"); args_parser.parse(arguments); - auto emoji_test_file = TRY(open_file(emoji_test_path, Core::Stream::OpenMode::Read)); + auto emoji_test_file = TRY(open_file(emoji_test_path, Core::File::OpenMode::Read)); VERIFY(!emoji_resource_path.is_empty() && Core::DeprecatedFile::exists(emoji_resource_path)); EmojiData emoji_data {}; TRY(parse_emoji_test_data(*emoji_test_file, emoji_data)); if (!emoji_serenity_path.is_empty()) { - auto emoji_serenity_file = TRY(open_file(emoji_serenity_path, Core::Stream::OpenMode::Read)); + auto emoji_serenity_file = TRY(open_file(emoji_serenity_path, Core::File::OpenMode::Read)); TRY(parse_emoji_serenity_data(*emoji_serenity_file, emoji_data)); } @@ -357,11 +357,11 @@ ErrorOr serenity_main(Main::Arguments arguments) } if (!generated_header_path.is_empty()) { - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); TRY(generate_emoji_data_header(*generated_header_file, emoji_data)); } if (!generated_implementation_path.is_empty()) { - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_emoji_data_implementation(*generated_implementation_file, emoji_data)); } @@ -371,7 +371,7 @@ ErrorOr serenity_main(Main::Arguments arguments) for (auto& emoji : emoji_data.emojis) set_image_path_for_emoji(emoji_resource_path, emoji); - auto generated_installation_file = TRY(open_file(generated_installation_path, Core::Stream::OpenMode::Write)); + auto generated_installation_file = TRY(open_file(generated_installation_path, Core::File::OpenMode::Write)); TRY(generate_emoji_installation(*generated_installation_file, emoji_data)); } diff --git a/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateUnicodeData.cpp b/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateUnicodeData.cpp index 3fc0c748088e14..f3244b3507dd55 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateUnicodeData.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GenerateUnicodeData.cpp @@ -219,7 +219,7 @@ static CodePointRange parse_code_point_range(StringView list) return code_point_range; } -static ErrorOr parse_special_casing(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr parse_special_casing(Core::BufferedFile& file, UnicodeData& unicode_data) { Array buffer; @@ -290,7 +290,7 @@ static ErrorOr parse_special_casing(Core::Stream::BufferedFile& file, Unic return {}; } -static ErrorOr parse_case_folding(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr parse_case_folding(Core::BufferedFile& file, UnicodeData& unicode_data) { Array buffer; @@ -338,7 +338,7 @@ static ErrorOr parse_case_folding(Core::Stream::BufferedFile& file, Unicod return {}; } -static ErrorOr parse_prop_list(Core::Stream::BufferedFile& file, PropList& prop_list, bool multi_value_property = false, bool sanitize_property = false) +static ErrorOr parse_prop_list(Core::BufferedFile& file, PropList& prop_list, bool multi_value_property = false, bool sanitize_property = false) { Array buffer; @@ -371,7 +371,7 @@ static ErrorOr parse_prop_list(Core::Stream::BufferedFile& file, PropList& return {}; } -static ErrorOr parse_alias_list(Core::Stream::BufferedFile& file, PropList const& prop_list, Vector& prop_aliases) +static ErrorOr parse_alias_list(Core::BufferedFile& file, PropList const& prop_list, Vector& prop_aliases) { DeprecatedString current_property; Array buffer; @@ -417,7 +417,7 @@ static ErrorOr parse_alias_list(Core::Stream::BufferedFile& file, PropList return {}; } -static ErrorOr parse_name_aliases(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr parse_name_aliases(Core::BufferedFile& file, UnicodeData& unicode_data) { Array buffer; @@ -448,7 +448,7 @@ static ErrorOr parse_name_aliases(Core::Stream::BufferedFile& file, Unicod return {}; } -static ErrorOr parse_value_alias_list(Core::Stream::BufferedFile& file, StringView desired_category, Vector const& value_list, Vector& prop_aliases, bool primary_value_is_first = true, bool sanitize_alias = false) +static ErrorOr parse_value_alias_list(Core::BufferedFile& file, StringView desired_category, Vector const& value_list, Vector& prop_aliases, bool primary_value_is_first = true, bool sanitize_alias = false) { TRY(file.seek(0, SeekMode::SetPosition)); Array buffer; @@ -494,7 +494,7 @@ static ErrorOr parse_value_alias_list(Core::Stream::BufferedFile& file, St return {}; } -static ErrorOr parse_normalization_props(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr parse_normalization_props(Core::BufferedFile& file, UnicodeData& unicode_data) { Array buffer; @@ -628,7 +628,7 @@ static Optional parse_decomposition_mapping(StringView s return mapping; } -static ErrorOr parse_block_display_names(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr parse_block_display_names(Core::BufferedFile& file, UnicodeData& unicode_data) { Array buffer; while (TRY(file.can_read_line())) { @@ -651,7 +651,7 @@ static ErrorOr parse_block_display_names(Core::Stream::BufferedFile& file, return {}; } -static ErrorOr parse_unicode_data(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr parse_unicode_data(Core::BufferedFile& file, UnicodeData& unicode_data) { Optional code_point_range_start; @@ -756,7 +756,7 @@ static ErrorOr parse_unicode_data(Core::Stream::BufferedFile& file, Unicod return {}; } -static ErrorOr generate_unicode_data_header(Core::Stream::BufferedFile& file, UnicodeData& unicode_data) +static ErrorOr generate_unicode_data_header(Core::BufferedFile& file, UnicodeData& unicode_data) { StringBuilder builder; SourceGenerator generator { builder }; @@ -871,7 +871,7 @@ ReadonlySpan case_folding_mapping(u32 code_point); return {}; } -static ErrorOr generate_unicode_data_implementation(Core::Stream::BufferedFile& file, UnicodeData const& unicode_data) +static ErrorOr generate_unicode_data_implementation(Core::BufferedFile& file, UnicodeData const& unicode_data) { StringBuilder builder; SourceGenerator generator { builder }; @@ -1530,26 +1530,26 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(sentence_break_path, "Path to SentenceBreakProperty.txt file", "sentence-break-path", 'i', "sentence-break-path"); args_parser.parse(arguments); - auto generated_header_file = TRY(open_file(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::Stream::OpenMode::Write)); - auto unicode_data_file = TRY(open_file(unicode_data_path, Core::Stream::OpenMode::Read)); - auto derived_general_category_file = TRY(open_file(derived_general_category_path, Core::Stream::OpenMode::Read)); - auto special_casing_file = TRY(open_file(special_casing_path, Core::Stream::OpenMode::Read)); - auto case_folding_file = TRY(open_file(case_folding_path, Core::Stream::OpenMode::Read)); - auto prop_list_file = TRY(open_file(prop_list_path, Core::Stream::OpenMode::Read)); - auto derived_core_prop_file = TRY(open_file(derived_core_prop_path, Core::Stream::OpenMode::Read)); - auto derived_binary_prop_file = TRY(open_file(derived_binary_prop_path, Core::Stream::OpenMode::Read)); - auto prop_alias_file = TRY(open_file(prop_alias_path, Core::Stream::OpenMode::Read)); - auto prop_value_alias_file = TRY(open_file(prop_value_alias_path, Core::Stream::OpenMode::Read)); - auto name_alias_file = TRY(open_file(name_alias_path, Core::Stream::OpenMode::Read)); - auto scripts_file = TRY(open_file(scripts_path, Core::Stream::OpenMode::Read)); - auto script_extensions_file = TRY(open_file(script_extensions_path, Core::Stream::OpenMode::Read)); - auto blocks_file = TRY(open_file(blocks_path, Core::Stream::OpenMode::Read)); - auto emoji_data_file = TRY(open_file(emoji_data_path, Core::Stream::OpenMode::Read)); - auto normalization_file = TRY(open_file(normalization_path, Core::Stream::OpenMode::Read)); - auto grapheme_break_file = TRY(open_file(grapheme_break_path, Core::Stream::OpenMode::Read)); - auto word_break_file = TRY(open_file(word_break_path, Core::Stream::OpenMode::Read)); - auto sentence_break_file = TRY(open_file(sentence_break_path, Core::Stream::OpenMode::Read)); + auto generated_header_file = TRY(open_file(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(open_file(generated_implementation_path, Core::File::OpenMode::Write)); + auto unicode_data_file = TRY(open_file(unicode_data_path, Core::File::OpenMode::Read)); + auto derived_general_category_file = TRY(open_file(derived_general_category_path, Core::File::OpenMode::Read)); + auto special_casing_file = TRY(open_file(special_casing_path, Core::File::OpenMode::Read)); + auto case_folding_file = TRY(open_file(case_folding_path, Core::File::OpenMode::Read)); + auto prop_list_file = TRY(open_file(prop_list_path, Core::File::OpenMode::Read)); + auto derived_core_prop_file = TRY(open_file(derived_core_prop_path, Core::File::OpenMode::Read)); + auto derived_binary_prop_file = TRY(open_file(derived_binary_prop_path, Core::File::OpenMode::Read)); + auto prop_alias_file = TRY(open_file(prop_alias_path, Core::File::OpenMode::Read)); + auto prop_value_alias_file = TRY(open_file(prop_value_alias_path, Core::File::OpenMode::Read)); + auto name_alias_file = TRY(open_file(name_alias_path, Core::File::OpenMode::Read)); + auto scripts_file = TRY(open_file(scripts_path, Core::File::OpenMode::Read)); + auto script_extensions_file = TRY(open_file(script_extensions_path, Core::File::OpenMode::Read)); + auto blocks_file = TRY(open_file(blocks_path, Core::File::OpenMode::Read)); + auto emoji_data_file = TRY(open_file(emoji_data_path, Core::File::OpenMode::Read)); + auto normalization_file = TRY(open_file(normalization_path, Core::File::OpenMode::Read)); + auto grapheme_break_file = TRY(open_file(grapheme_break_path, Core::File::OpenMode::Read)); + auto word_break_file = TRY(open_file(word_break_path, Core::File::OpenMode::Read)); + auto sentence_break_file = TRY(open_file(sentence_break_path, Core::File::OpenMode::Read)); UnicodeData unicode_data {}; TRY(parse_special_casing(*special_casing_file, unicode_data)); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GeneratorUtil.h b/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GeneratorUtil.h index c83378f6f94409..472d489a61bec1 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GeneratorUtil.h +++ b/Meta/Lagom/Tools/CodeGenerators/LibUnicode/GeneratorUtil.h @@ -20,6 +20,7 @@ #include #include #include +#include #include #include @@ -323,18 +324,18 @@ struct CanonicalLanguageID { Vector variants {}; }; -inline ErrorOr> open_file(StringView path, Core::Stream::OpenMode mode) +inline ErrorOr> open_file(StringView path, Core::File::OpenMode mode) { if (path.is_empty()) return Error::from_string_literal("Provided path is empty, please provide all command line options"); - auto file = TRY(Core::Stream::File::open(path, mode)); - return Core::Stream::BufferedFile::create(move(file)); + auto file = TRY(Core::File::open(path, mode)); + return Core::BufferedFile::create(move(file)); } inline ErrorOr read_json_file(StringView path) { - auto file = TRY(open_file(path, Core::Stream::OpenMode::Read)); + auto file = TRY(open_file(path, Core::File::OpenMode::Read)); auto buffer = TRY(file->read_until_eof()); return JsonValue::from_string(buffer); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/BindingsGenerator/main.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/BindingsGenerator/main.cpp index ec07396c5f015d..86f8a6031f1336 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/BindingsGenerator/main.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/BindingsGenerator/main.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -64,7 +65,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(import_base_path, "Import base path", "import-base-path", Core::ArgsParser::Required::No); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); LexicalPath lexical_path(path); auto& namespace_ = lexical_path.parts_view().at(lexical_path.parts_view().size() - 2); @@ -74,7 +75,7 @@ ErrorOr serenity_main(Main::Arguments arguments) if (import_base_path.is_null()) import_base_path = lexical_path.dirname(); - auto output_file = TRY(Core::Stream::File::open_file_or_standard_stream(output_path, Core::Stream::OpenMode::Write)); + auto output_file = TRY(Core::File::open_file_or_standard_stream(output_path, Core::File::OpenMode::Write)); IDL::Parser parser(path, data, import_base_path); auto& interface = parser.parse(); @@ -151,7 +152,7 @@ ErrorOr serenity_main(Main::Arguments arguments) TRY(output_file->write(output_builder.string_view().bytes())); if (!depfile_path.is_null()) { - auto depfile = TRY(Core::Stream::File::open_file_or_standard_stream(depfile_path, Core::Stream::OpenMode::Write)); + auto depfile = TRY(Core::File::open_file_or_standard_stream(depfile_path, Core::File::OpenMode::Write)); StringBuilder depfile_builder; depfile_builder.append(depfile_target.is_null() ? output_path : depfile_target); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSEnums.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSEnums.cpp index 891d43f0bcc161..51e21dbbf44dd4 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSEnums.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSEnums.cpp @@ -10,8 +10,8 @@ #include #include -ErrorOr generate_header_file(JsonObject& enums_data, Core::Stream::File& file); -ErrorOr generate_implementation_file(JsonObject& enums_data, Core::Stream::File& file); +ErrorOr generate_header_file(JsonObject& enums_data, Core::File& file); +ErrorOr generate_implementation_file(JsonObject& enums_data, Core::File& file); ErrorOr serenity_main(Main::Arguments arguments) { @@ -29,8 +29,8 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(json.is_object()); auto enums_data = json.as_object(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_header_file(enums_data, *generated_header_file)); TRY(generate_implementation_file(enums_data, *generated_implementation_file)); @@ -38,7 +38,7 @@ ErrorOr serenity_main(Main::Arguments arguments) return 0; } -ErrorOr generate_header_file(JsonObject& enums_data, Core::Stream::File& file) +ErrorOr generate_header_file(JsonObject& enums_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -99,7 +99,7 @@ enum class ValueID; return {}; } -ErrorOr generate_implementation_file(JsonObject& enums_data, Core::Stream::File& file) +ErrorOr generate_implementation_file(JsonObject& enums_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSMediaFeatureID.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSMediaFeatureID.cpp index 11b7f084233871..6cf5abcf43bc20 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSMediaFeatureID.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSMediaFeatureID.cpp @@ -10,8 +10,8 @@ #include #include -ErrorOr generate_header_file(JsonObject& media_feature_data, Core::Stream::File& file); -ErrorOr generate_implementation_file(JsonObject& media_feature_data, Core::Stream::File& file); +ErrorOr generate_header_file(JsonObject& media_feature_data, Core::File& file); +ErrorOr generate_implementation_file(JsonObject& media_feature_data, Core::File& file); ErrorOr serenity_main(Main::Arguments arguments) { @@ -29,8 +29,8 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(json.is_object()); auto media_feature_data = json.as_object(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_header_file(media_feature_data, *generated_header_file)); TRY(generate_implementation_file(media_feature_data, *generated_implementation_file)); @@ -38,7 +38,7 @@ ErrorOr serenity_main(Main::Arguments arguments) return 0; } -ErrorOr generate_header_file(JsonObject& media_feature_data, Core::Stream::File& file) +ErrorOr generate_header_file(JsonObject& media_feature_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -84,7 +84,7 @@ bool media_feature_accepts_identifier(MediaFeatureID, ValueID); return {}; } -ErrorOr generate_implementation_file(JsonObject& media_feature_data, Core::Stream::File& file) +ErrorOr generate_implementation_file(JsonObject& media_feature_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSPropertyID.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSPropertyID.cpp index ad90d396a6defb..5724dd7b1cb8ea 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSPropertyID.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSPropertyID.cpp @@ -11,8 +11,8 @@ #include #include -ErrorOr generate_header_file(JsonObject& properties, Core::Stream::File& file); -ErrorOr generate_implementation_file(JsonObject& properties, Core::Stream::File& file); +ErrorOr generate_header_file(JsonObject& properties, Core::File& file); +ErrorOr generate_implementation_file(JsonObject& properties, Core::File& file); ErrorOr serenity_main(Main::Arguments arguments) { @@ -30,8 +30,8 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(json.is_object()); auto properties = json.as_object(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_header_file(properties, *generated_header_file)); TRY(generate_implementation_file(properties, *generated_implementation_file)); @@ -39,7 +39,7 @@ ErrorOr serenity_main(Main::Arguments arguments) return 0; } -ErrorOr generate_header_file(JsonObject& properties, Core::Stream::File& file) +ErrorOr generate_header_file(JsonObject& properties, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -143,7 +143,7 @@ struct Traits : public GenericTraits return {}; } -ErrorOr generate_implementation_file(JsonObject& properties, Core::Stream::File& file) +ErrorOr generate_implementation_file(JsonObject& properties, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSTransformFunctions.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSTransformFunctions.cpp index 4dedf3e81fe3c1..9276061b12a599 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSTransformFunctions.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSTransformFunctions.cpp @@ -11,8 +11,8 @@ #include #include -ErrorOr generate_header_file(JsonObject& transforms_data, Core::Stream::File& file); -ErrorOr generate_implementation_file(JsonObject& transforms_data, Core::Stream::File& file); +ErrorOr generate_header_file(JsonObject& transforms_data, Core::File& file); +ErrorOr generate_implementation_file(JsonObject& transforms_data, Core::File& file); ErrorOr serenity_main(Main::Arguments arguments) { @@ -30,8 +30,8 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(json.is_object()); auto transforms_data = json.as_object(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_header_file(transforms_data, *generated_header_file)); TRY(generate_implementation_file(transforms_data, *generated_implementation_file)); @@ -48,7 +48,7 @@ static DeprecatedString title_casify_transform_function(StringView input) return builder.to_deprecated_string(); } -ErrorOr generate_header_file(JsonObject& transforms_data, Core::Stream::File& file) +ErrorOr generate_header_file(JsonObject& transforms_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -100,7 +100,7 @@ TransformFunctionMetadata transform_function_metadata(TransformFunction); return {}; } -ErrorOr generate_implementation_file(JsonObject& transforms_data, Core::Stream::File& file) +ErrorOr generate_implementation_file(JsonObject& transforms_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSValueID.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSValueID.cpp index bb70156c6d50ce..ba0dba9088ce6e 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSValueID.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateCSSValueID.cpp @@ -11,8 +11,8 @@ #include #include -ErrorOr generate_header_file(JsonArray& identifier_data, Core::Stream::File& file); -ErrorOr generate_implementation_file(JsonArray& identifier_data, Core::Stream::File& file); +ErrorOr generate_header_file(JsonArray& identifier_data, Core::File& file); +ErrorOr generate_implementation_file(JsonArray& identifier_data, Core::File& file); ErrorOr serenity_main(Main::Arguments arguments) { @@ -30,8 +30,8 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(json.is_array()); auto identifier_data = json.as_array(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generate_header_file(identifier_data, *generated_header_file)); TRY(generate_implementation_file(identifier_data, *generated_implementation_file)); @@ -39,7 +39,7 @@ ErrorOr serenity_main(Main::Arguments arguments) return 0; } -ErrorOr generate_header_file(JsonArray& identifier_data, Core::Stream::File& file) +ErrorOr generate_header_file(JsonArray& identifier_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; @@ -78,7 +78,7 @@ StringView string_from_value_id(ValueID); return {}; } -ErrorOr generate_implementation_file(JsonArray& identifier_data, Core::Stream::File& file) +ErrorOr generate_implementation_file(JsonArray& identifier_data, Core::File& file) { StringBuilder builder; SourceGenerator generator { builder }; diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateWindowOrWorkerInterfaces.cpp b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateWindowOrWorkerInterfaces.cpp index 03e3ee7b313bcc..09806e93a431f5 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateWindowOrWorkerInterfaces.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GenerateWindowOrWorkerInterfaces.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -110,7 +111,7 @@ class @legacy_constructor_class@;)~~~"); )~~~"); auto generated_forward_path = LexicalPath(output_path).append("Forward.h"sv).string(); - auto generated_forward_file = TRY(Core::Stream::File::open(generated_forward_path, Core::Stream::OpenMode::Write)); + auto generated_forward_file = TRY(Core::File::open(generated_forward_path, Core::File::OpenMode::Write)); TRY(generated_forward_file->write(generator.as_string_view().bytes())); return {}; @@ -224,7 +225,7 @@ void Intrinsics::create_web_prototype_and_constructor<@prototype_class@>(JS::Rea )~~~"); auto generated_intrinsics_path = LexicalPath(output_path).append("IntrinsicDefinitions.cpp"sv).string(); - auto generated_intrinsics_file = TRY(Core::Stream::File::open(generated_intrinsics_path, Core::Stream::OpenMode::Write)); + auto generated_intrinsics_file = TRY(Core::File::open(generated_intrinsics_path, Core::File::OpenMode::Write)); TRY(generated_intrinsics_file->write(generator.as_string_view().bytes())); return {}; @@ -250,7 +251,7 @@ void add_@global_object_snake_name@_exposed_interfaces(JS::Object&); )~~~"); auto generated_header_path = LexicalPath(output_path).append(DeprecatedString::formatted("{}ExposedInterfaces.h", class_name)).string(); - auto generated_header_file = TRY(Core::Stream::File::open(generated_header_path, Core::Stream::OpenMode::Write)); + auto generated_header_file = TRY(Core::File::open(generated_header_path, Core::File::OpenMode::Write)); TRY(generated_header_file->write(generator.as_string_view().bytes())); return {}; @@ -332,7 +333,7 @@ void add_@global_object_snake_name@_exposed_interfaces(JS::Object& global) )~~~"); auto generated_implementation_path = LexicalPath(output_path).append(DeprecatedString::formatted("{}ExposedInterfaces.cpp", class_name)).string(); - auto generated_implementation_file = TRY(Core::Stream::File::open(generated_implementation_path, Core::Stream::OpenMode::Write)); + auto generated_implementation_file = TRY(Core::File::open(generated_implementation_path, Core::File::OpenMode::Write)); TRY(generated_implementation_file->write(generator.as_string_view().bytes())); return {}; @@ -359,7 +360,7 @@ ErrorOr serenity_main(Main::Arguments arguments) // Read in all IDL files, we must own the storage for all of these for the lifetime of the program Vector file_contents; for (DeprecatedString const& path : paths) { - auto file_or_error = Core::Stream::File::open(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(path, Core::File::OpenMode::Read); if (file_or_error.is_error()) { s_error_string = DeprecatedString::formatted("Unable to open file {}", path); return Error::from_string_view(s_error_string); diff --git a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GeneratorUtil.h b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GeneratorUtil.h index f3b396a7566d05..5fa86d90ae7c50 100644 --- a/Meta/Lagom/Tools/CodeGenerators/LibWeb/GeneratorUtil.h +++ b/Meta/Lagom/Tools/CodeGenerators/LibWeb/GeneratorUtil.h @@ -10,6 +10,7 @@ #include #include #include +#include #include #include @@ -56,7 +57,7 @@ DeprecatedString snake_casify(DeprecatedString const& dashy_name) ErrorOr read_entire_file_as_json(StringView filename) { - auto file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(filename, Core::File::OpenMode::Read)); auto json_size = TRY(file->size()); auto json_data = TRY(ByteBuffer::create_uninitialized(json_size)); TRY(file->read_entire_buffer(json_data.bytes())); diff --git a/Meta/Lagom/Tools/CodeGenerators/StateMachineGenerator/main.cpp b/Meta/Lagom/Tools/CodeGenerators/StateMachineGenerator/main.cpp index 984e4eff59ac6c..f3755f68c6d838 100644 --- a/Meta/Lagom/Tools/CodeGenerators/StateMachineGenerator/main.cpp +++ b/Meta/Lagom/Tools/CodeGenerators/StateMachineGenerator/main.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -221,7 +222,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(path, "Path to parser description", "input", Core::ArgsParser::Required::Yes); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto content = TRY(file->read_until_eof()); auto state_machine = parse_state_machine(content); diff --git a/Meta/Lagom/Tools/ConfigureComponents/main.cpp b/Meta/Lagom/Tools/ConfigureComponents/main.cpp index a184dd5405ed4d..e356589b0cf1bc 100644 --- a/Meta/Lagom/Tools/ConfigureComponents/main.cpp +++ b/Meta/Lagom/Tools/ConfigureComponents/main.cpp @@ -192,7 +192,7 @@ static Result, int> run_whiptail(WhiptailMode mode, Vec return return_code; } - auto file_or_error = Core::Stream::File::adopt_fd(read_fd, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::adopt_fd(read_fd, Core::File::OpenMode::Read); if (file_or_error.is_error()) { warnln("\e[31mError:\e[0m Could not adopt file descriptor for reading: {}", file_or_error.error()); return -1; diff --git a/Ports/OpenJDK/patches/0008-java.base-Enable-java.lang.Process-on-serenity.patch b/Ports/OpenJDK/patches/0008-java.base-Enable-java.lang.Process-on-serenity.patch index 50763e01cee3cc..15fd209e626d11 100644 --- a/Ports/OpenJDK/patches/0008-java.base-Enable-java.lang.Process-on-serenity.patch +++ b/Ports/OpenJDK/patches/0008-java.base-Enable-java.lang.Process-on-serenity.patch @@ -50,7 +50,7 @@ new file mode 100644 index 0000000000000000000000000000000000000000..cc0c08cb85a682d66a00f6b48ad2871f83b5e719 --- /dev/null +++ b/src/java.base/serenity/native/libjava/ProcessHandleImpl_serenity.cpp -@@ -0,0 +1,165 @@ +@@ -0,0 +1,166 @@ +/* + * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved. + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. @@ -89,6 +89,7 @@ index 0000000000000000000000000000000000000000..cc0c08cb85a682d66a00f6b48ad2871f + +#include +#include ++#include +#include +#include +#include @@ -189,7 +190,7 @@ index 0000000000000000000000000000000000000000..cc0c08cb85a682d66a00f6b48ad2871f + unix_getUserInfo(env, jinfo, proc.pid); + JNU_CHECK_EXCEPTION(env); + -+ auto cmdline_file = JAVA_TRY(Core::Stream::File::open(DeprecatedString::formatted("/proc/{}/cmdline", pid), Core::Stream::OpenMode::Read), "Unable to open /proc/pid/cmdline"sv); ++ auto cmdline_file = JAVA_TRY(Core::File::open(DeprecatedString::formatted("/proc/{}/cmdline", pid), Core::File::OpenMode::Read), "Unable to open /proc/pid/cmdline"sv); + auto contents = JAVA_TRY(cmdline_file->read_until_eof(), "Unable to read /proc/pid/cmdline"sv); + auto cmdline = JAVA_TRY(JsonValue::from_string(contents), "Invalid JSON in /proc/pid/cmdline"sv); + diff --git a/Tests/LibCompress/TestBrotli.cpp b/Tests/LibCompress/TestBrotli.cpp index 3acedb460e8175..f4683d322d1ec8 100644 --- a/Tests/LibCompress/TestBrotli.cpp +++ b/Tests/LibCompress/TestBrotli.cpp @@ -7,6 +7,7 @@ #include #include +#include #include static void run_test(StringView const file_name) @@ -18,12 +19,12 @@ static void run_test(StringView const file_name) DeprecatedString path = DeprecatedString::formatted("brotli-test-files/{}", file_name); #endif - auto cmp_file = MUST(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto cmp_file = MUST(Core::File::open(path, Core::File::OpenMode::Read)); auto cmp_data = MUST(cmp_file->read_until_eof()); DeprecatedString path_compressed = DeprecatedString::formatted("{}.br", path); - auto file = MUST(Core::Stream::File::open(path_compressed, Core::Stream::OpenMode::Read)); + auto file = MUST(Core::File::open(path_compressed, Core::File::OpenMode::Read)); auto brotli_stream = Compress::BrotliDecompressionStream { *file }; auto data = MUST(brotli_stream.read_until_eof()); @@ -96,7 +97,7 @@ TEST_CASE(brotli_decompress_zero_one_bin) DeprecatedString path_compressed = DeprecatedString::formatted("{}.br", path); - auto file = MUST(Core::Stream::File::open(path_compressed, Core::Stream::OpenMode::Read)); + auto file = MUST(Core::File::open(path_compressed, Core::File::OpenMode::Read)); auto brotli_stream = Compress::BrotliDecompressionStream { *file }; u8 buffer_raw[4096]; diff --git a/Tests/LibCore/TestLibCoreStream.cpp b/Tests/LibCore/TestLibCoreStream.cpp index 895b77434f897b..861711be8648a0 100644 --- a/Tests/LibCore/TestLibCoreStream.cpp +++ b/Tests/LibCore/TestLibCoreStream.cpp @@ -8,6 +8,7 @@ #include #include #include +#include #include #include #include @@ -23,7 +24,7 @@ TEST_CASE(file_open) { - auto maybe_file = Core::Stream::File::open("/tmp/file-open-test.txt"sv, Core::Stream::OpenMode::Write); + auto maybe_file = Core::File::open("/tmp/file-open-test.txt"sv, Core::File::OpenMode::Write); if (maybe_file.is_error()) { warnln("Failed to open the file: {}", strerror(maybe_file.error().code())); VERIFY_NOT_REACHED(); @@ -41,7 +42,7 @@ TEST_CASE(file_open) TEST_CASE(file_write_bytes) { - auto maybe_file = Core::Stream::File::open("/tmp/file-write-bytes-test.txt"sv, Core::Stream::OpenMode::Write); + auto maybe_file = Core::File::open("/tmp/file-write-bytes-test.txt"sv, Core::File::OpenMode::Write); auto file = maybe_file.release_value(); constexpr auto some_words = "These are some words"sv; @@ -54,7 +55,7 @@ constexpr auto expected_buffer_contents = "<small>(Please consider transla TEST_CASE(file_read_bytes) { - auto maybe_file = Core::Stream::File::open("/usr/Tests/LibCore/long_lines.txt"sv, Core::Stream::OpenMode::Read); + auto maybe_file = Core::File::open("/usr/Tests/LibCore/long_lines.txt"sv, Core::File::OpenMode::Read); EXPECT(!maybe_file.is_error()); auto file = maybe_file.release_value(); @@ -76,7 +77,7 @@ constexpr auto expected_seek_contents3 = "levels of advanc"sv; TEST_CASE(file_seeking_around) { - auto maybe_file = Core::Stream::File::open("/usr/Tests/LibCore/long_lines.txt"sv, Core::Stream::OpenMode::Read); + auto maybe_file = Core::File::open("/usr/Tests/LibCore/long_lines.txt"sv, Core::File::OpenMode::Read); EXPECT(!maybe_file.is_error()); auto file = maybe_file.release_value(); @@ -109,7 +110,7 @@ TEST_CASE(file_adopt_fd) int rc = ::open("/usr/Tests/LibCore/long_lines.txt", O_RDONLY); EXPECT(rc >= 0); - auto maybe_file = Core::Stream::File::adopt_fd(rc, Core::Stream::OpenMode::Read); + auto maybe_file = Core::File::adopt_fd(rc, Core::File::OpenMode::Read); EXPECT(!maybe_file.is_error()); auto file = maybe_file.release_value(); @@ -131,14 +132,14 @@ TEST_CASE(file_adopt_fd) TEST_CASE(file_adopt_invalid_fd) { - auto maybe_file = Core::Stream::File::adopt_fd(-1, Core::Stream::OpenMode::Read); + auto maybe_file = Core::File::adopt_fd(-1, Core::File::OpenMode::Read); EXPECT(maybe_file.is_error()); EXPECT_EQ(maybe_file.error().code(), EBADF); } TEST_CASE(file_truncate) { - auto maybe_file = Core::Stream::File::open("/tmp/file-truncate-test.txt"sv, Core::Stream::OpenMode::Write); + auto maybe_file = Core::File::open("/tmp/file-truncate-test.txt"sv, Core::File::OpenMode::Write); auto file = maybe_file.release_value(); EXPECT(!file->truncate(999).is_error()); @@ -420,9 +421,9 @@ TEST_CASE(local_socket_write) TEST_CASE(buffered_long_file_read) { - auto maybe_file = Core::Stream::File::open("/usr/Tests/LibCore/long_lines.txt"sv, Core::Stream::OpenMode::Read); + auto maybe_file = Core::File::open("/usr/Tests/LibCore/long_lines.txt"sv, Core::File::OpenMode::Read); EXPECT(!maybe_file.is_error()); - auto maybe_buffered_file = Core::Stream::BufferedFile::create(maybe_file.release_value()); + auto maybe_buffered_file = Core::BufferedFile::create(maybe_file.release_value()); EXPECT(!maybe_buffered_file.is_error()); auto file = maybe_buffered_file.release_value(); @@ -442,9 +443,9 @@ TEST_CASE(buffered_long_file_read) TEST_CASE(buffered_small_file_read) { - auto maybe_file = Core::Stream::File::open("/usr/Tests/LibCore/small.txt"sv, Core::Stream::OpenMode::Read); + auto maybe_file = Core::File::open("/usr/Tests/LibCore/small.txt"sv, Core::File::OpenMode::Read); EXPECT(!maybe_file.is_error()); - auto maybe_buffered_file = Core::Stream::BufferedFile::create(maybe_file.release_value()); + auto maybe_buffered_file = Core::BufferedFile::create(maybe_file.release_value()); EXPECT(!maybe_buffered_file.is_error()); auto file = maybe_buffered_file.release_value(); @@ -471,8 +472,8 @@ TEST_CASE(buffered_small_file_read) TEST_CASE(buffered_file_tell_and_seek) { // We choose a buffer size of 12 bytes to cover half of the input file. - auto file = Core::Stream::File::open("/usr/Tests/LibCore/small.txt"sv, Core::Stream::OpenMode::Read).release_value(); - auto buffered_file = Core::Stream::BufferedFile::create(move(file), 12).release_value(); + auto file = Core::File::open("/usr/Tests/LibCore/small.txt"sv, Core::File::OpenMode::Read).release_value(); + auto buffered_file = Core::BufferedFile::create(move(file), 12).release_value(); // Initial state. { diff --git a/Tests/LibCpp/test-cpp-parser.cpp b/Tests/LibCpp/test-cpp-parser.cpp index 681368d2cae7af..e3d0796b9012b2 100644 --- a/Tests/LibCpp/test-cpp-parser.cpp +++ b/Tests/LibCpp/test-cpp-parser.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -15,7 +16,7 @@ constexpr char TESTS_ROOT_DIR[] = "/home/anon/Tests/cpp-tests/parser"; static DeprecatedString read_all(DeprecatedString const& path) { - auto file = MUST(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = MUST(Core::File::open(path, Core::File::OpenMode::Read)); auto file_size = MUST(file->size()); auto content = MUST(ByteBuffer::create_uninitialized(file_size)); MUST(file->read_entire_buffer(content.bytes())); diff --git a/Tests/LibCpp/test-cpp-preprocessor.cpp b/Tests/LibCpp/test-cpp-preprocessor.cpp index 924d5c4d538869..14b7cfc9b14a1d 100644 --- a/Tests/LibCpp/test-cpp-preprocessor.cpp +++ b/Tests/LibCpp/test-cpp-preprocessor.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -14,7 +15,7 @@ constexpr char TESTS_ROOT_DIR[] = "/home/anon/Tests/cpp-tests/preprocessor"; static DeprecatedString read_all(DeprecatedString const& path) { - auto file = MUST(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = MUST(Core::File::open(path, Core::File::OpenMode::Read)); auto file_size = MUST(file->size()); auto content = MUST(ByteBuffer::create_uninitialized(file_size)); MUST(file->read_entire_buffer(content.bytes())); diff --git a/Tests/LibGL/TestRender.cpp b/Tests/LibGL/TestRender.cpp index 8fcb7fe3f012d3..11a19492085ad6 100644 --- a/Tests/LibGL/TestRender.cpp +++ b/Tests/LibGL/TestRender.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include #include @@ -36,7 +37,7 @@ static void expect_bitmap_equals_reference(Gfx::Bitmap const& bitmap, StringView if constexpr (SAVE_OUTPUT) { auto target_path = LexicalPath("/home/anon").append(reference_filename); auto qoi_buffer = Gfx::QOIWriter::encode(bitmap); - auto qoi_output_stream = MUST(Core::Stream::File::open(target_path.string(), Core::Stream::OpenMode::Write)); + auto qoi_output_stream = MUST(Core::File::open(target_path.string(), Core::File::OpenMode::Write)); MUST(qoi_output_stream->write_entire_buffer(qoi_buffer)); } diff --git a/Tests/LibJS/test-test262.cpp b/Tests/LibJS/test-test262.cpp index 075197bbb79f1b..bc90d888deaf74 100644 --- a/Tests/LibJS/test-test262.cpp +++ b/Tests/LibJS/test-test262.cpp @@ -14,6 +14,7 @@ #include #include #include +#include #include #include #include @@ -131,16 +132,16 @@ class Test262RunnerHandler { TRY(Core::System::close(write_pipe_fds[0])); TRY(Core::System::close(read_pipe_fds[1])); - auto infile = TRY(Core::Stream::File::adopt_fd(read_pipe_fds[0], Core::Stream::OpenMode::Read)); + auto infile = TRY(Core::File::adopt_fd(read_pipe_fds[0], Core::File::OpenMode::Read)); - auto outfile = TRY(Core::Stream::File::adopt_fd(write_pipe_fds[1], Core::Stream::OpenMode::Write)); + auto outfile = TRY(Core::File::adopt_fd(write_pipe_fds[1], Core::File::OpenMode::Write)); runner_kill.disarm(); return make(pid, move(infile), move(outfile)); } - Test262RunnerHandler(pid_t pid, NonnullOwnPtr in_file, NonnullOwnPtr out_file) + Test262RunnerHandler(pid_t pid, NonnullOwnPtr in_file, NonnullOwnPtr out_file) : m_pid(pid) , m_input(move(in_file)) , m_output(move(out_file)) @@ -218,8 +219,8 @@ class Test262RunnerHandler { public: pid_t m_pid; - NonnullOwnPtr m_input; - NonnullOwnPtr m_output; + NonnullOwnPtr m_input; + NonnullOwnPtr m_output; }; static ErrorOr> run_test_files(Span files, size_t offset, StringView command, char const* const arguments[]) @@ -411,7 +412,7 @@ ErrorOr serenity_main(Main::Arguments arguments) void write_per_file(HashMap const& result_map, Vector const& paths, StringView per_file_name, double time_taken_in_ms) { - auto file_or_error = Core::Stream::File::open(per_file_name, Core::Stream::OpenMode::Write); + auto file_or_error = Core::File::open(per_file_name, Core::File::OpenMode::Write); if (file_or_error.is_error()) { warnln("Failed to open per file for writing at {}: {}", per_file_name, file_or_error.error().string_literal()); return; diff --git a/Tests/LibJS/test262-runner.cpp b/Tests/LibJS/test262-runner.cpp index 8511c06211b2b3..0e852e7cfbe5f7 100644 --- a/Tests/LibJS/test262-runner.cpp +++ b/Tests/LibJS/test262-runner.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -139,7 +140,7 @@ static Result read_harness_file(StringView harness_file) { auto cache = s_cached_harness_files.find(harness_file); if (cache == s_cached_harness_files.end()) { - auto file_or_error = Core::Stream::File::open(DeprecatedString::formatted("{}{}", s_harness_file_directory, harness_file), Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(DeprecatedString::formatted("{}{}", s_harness_file_directory, harness_file), Core::File::OpenMode::Read); if (file_or_error.is_error()) { return TestError { NegativePhase::Harness, @@ -679,12 +680,12 @@ int main(int argc, char** argv) #define DISARM_TIMER() \ alarm(0) - auto standard_input_or_error = Core::Stream::File::standard_input(); + auto standard_input_or_error = Core::File::standard_input(); if (standard_input_or_error.is_error()) return exit_setup_input_failure; Array input_buffer {}; - auto buffered_standard_input_or_error = Core::Stream::BufferedFile::create(standard_input_or_error.release_value()); + auto buffered_standard_input_or_error = Core::BufferedFile::create(standard_input_or_error.release_value()); if (buffered_standard_input_or_error.is_error()) return exit_setup_input_failure; @@ -708,7 +709,7 @@ int main(int argc, char** argv) VERIFY(!s_harness_file_directory.is_empty()); } - auto file_or_error = Core::Stream::File::open(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(path, Core::File::OpenMode::Read); if (file_or_error.is_error()) { warnln("Could not open file: {}", path); return exit_read_file_failure; diff --git a/Tests/LibMarkdown/TestCommonmark.cpp b/Tests/LibMarkdown/TestCommonmark.cpp index a9ef51540451b7..338bc56c116d87 100644 --- a/Tests/LibMarkdown/TestCommonmark.cpp +++ b/Tests/LibMarkdown/TestCommonmark.cpp @@ -8,6 +8,7 @@ #include #include #include +#include #include #include #include @@ -15,9 +16,9 @@ TEST_SETUP { - auto file_or_error = Core::Stream::File::open("/home/anon/Tests/commonmark.spec.json"sv, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open("/home/anon/Tests/commonmark.spec.json"sv, Core::File::OpenMode::Read); if (file_or_error.is_error()) - file_or_error = Core::Stream::File::open("./commonmark.spec.json"sv, Core::Stream::OpenMode::Read); + file_or_error = Core::File::open("./commonmark.spec.json"sv, Core::File::OpenMode::Read); VERIFY(!file_or_error.is_error()); auto file = file_or_error.release_value(); auto file_size = MUST(file->size()); diff --git a/Tests/LibWasm/test-wasm.cpp b/Tests/LibWasm/test-wasm.cpp index 1cbeff367a6d40..fce9d807e719cc 100644 --- a/Tests/LibWasm/test-wasm.cpp +++ b/Tests/LibWasm/test-wasm.cpp @@ -17,7 +17,7 @@ TESTJS_GLOBAL_FUNCTION(read_binary_wasm_file, readBinaryWasmFile) { auto& realm = *vm.current_realm(); auto filename = TRY(vm.argument(0).to_deprecated_string(vm)); - auto file = Core::Stream::File::open(filename, Core::Stream::OpenMode::Read); + auto file = Core::File::open(filename, Core::File::OpenMode::Read); if (file.is_error()) return vm.throw_completion(strerror(file.error().code())); diff --git a/Tests/LibWeb/TestHTMLTokenizer.cpp b/Tests/LibWeb/TestHTMLTokenizer.cpp index 977fd926bf9d11..1be1b0c6f8970c 100644 --- a/Tests/LibWeb/TestHTMLTokenizer.cpp +++ b/Tests/LibWeb/TestHTMLTokenizer.cpp @@ -6,6 +6,7 @@ #include +#include #include #include @@ -208,7 +209,7 @@ TEST_CASE(regression) StringView path = "tokenizer-test.html"sv; #endif - auto file = MUST(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = MUST(Core::File::open(path, Core::File::OpenMode::Read)); auto file_size = MUST(file->size()); auto content = MUST(ByteBuffer::create_uninitialized(file_size)); MUST(file->read(content.bytes())); diff --git a/Userland/Applets/Network/main.cpp b/Userland/Applets/Network/main.cpp index db637627ec6bda..88d66462983d6a 100644 --- a/Userland/Applets/Network/main.cpp +++ b/Userland/Applets/Network/main.cpp @@ -6,6 +6,7 @@ */ #include +#include #include #include #include @@ -107,7 +108,7 @@ class NetworkWidget final : public GUI::ImageWidget { { StringBuilder adapter_info; - auto file_or_error = Core::Stream::File::open("/sys/kernel/net/adapters"sv, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open("/sys/kernel/net/adapters"sv, Core::File::OpenMode::Read); if (file_or_error.is_error()) { dbgln("Error: Could not open /sys/kernel/net/adapters: {}", file_or_error.error()); return ""; diff --git a/Userland/Applets/ResourceGraph/main.cpp b/Userland/Applets/ResourceGraph/main.cpp index 8ba289d93703f1..2716c2ad96b33a 100644 --- a/Userland/Applets/ResourceGraph/main.cpp +++ b/Userland/Applets/ResourceGraph/main.cpp @@ -144,13 +144,13 @@ class GraphWidget final : public GUI::Frame { GUI::Process::spawn_or_show_error(window(), "/bin/SystemMonitor"sv, Array { "-t", m_graph_type == GraphType::Network ? "network" : "graphs" }); } - ErrorOr get_data_as_json(OwnPtr& file, StringView filename) + ErrorOr get_data_as_json(OwnPtr& file, StringView filename) { if (file) { // Seeking to the beginning causes a data refresh! TRY(file->seek(0, SeekMode::SetPosition)); } else { - file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Read)); + file = TRY(Core::File::open(filename, Core::File::OpenMode::Read)); } auto file_contents = TRY(file->read_until_eof()); @@ -231,9 +231,9 @@ class GraphWidget final : public GUI::Frame { static constexpr u64 const scale_unit = 8000; u64 m_current_scale { scale_unit }; DeprecatedString m_tooltip; - OwnPtr m_proc_stat; - OwnPtr m_proc_mem; - OwnPtr m_proc_net; + OwnPtr m_proc_stat; + OwnPtr m_proc_mem; + OwnPtr m_proc_net; }; ErrorOr serenity_main(Main::Arguments arguments) diff --git a/Userland/Applications/Browser/BrowserWindow.cpp b/Userland/Applications/Browser/BrowserWindow.cpp index 46931066eaf942..1f73a045e965b8 100644 --- a/Userland/Applications/Browser/BrowserWindow.cpp +++ b/Userland/Applications/Browser/BrowserWindow.cpp @@ -488,7 +488,7 @@ ErrorOr BrowserWindow::load_search_engines(GUI::Menu& settings_menu) m_search_engine_actions.add_action(*m_disable_search_engine_action); m_disable_search_engine_action->set_checked(true); - auto search_engines_file = TRY(Core::Stream::File::open(Browser::search_engines_file_path(), Core::Stream::OpenMode::Read)); + auto search_engines_file = TRY(Core::File::open(Browser::search_engines_file_path(), Core::File::OpenMode::Read)); auto file_size = TRY(search_engines_file->size()); auto buffer = TRY(ByteBuffer::create_uninitialized(file_size)); if (!search_engines_file->read_entire_buffer(buffer).is_error()) { @@ -775,7 +775,7 @@ ErrorOr BrowserWindow::take_screenshot(ScreenshotType type) auto encoded = TRY(Gfx::PNGWriter::encode(*bitmap.bitmap())); - auto screenshot_file = TRY(Core::Stream::File::open(path.string(), Core::Stream::OpenMode::Write)); + auto screenshot_file = TRY(Core::File::open(path.string(), Core::File::OpenMode::Write)); TRY(screenshot_file->write(encoded)); return {}; diff --git a/Userland/Applications/Browser/DownloadWidget.cpp b/Userland/Applications/Browser/DownloadWidget.cpp index db3a1fab8dffd1..ab6023cf71a618 100644 --- a/Userland/Applications/Browser/DownloadWidget.cpp +++ b/Userland/Applications/Browser/DownloadWidget.cpp @@ -47,7 +47,7 @@ DownloadWidget::DownloadWidget(const URL& url) }; { - auto file_or_error = Core::Stream::File::open(m_destination_path, Core::Stream::OpenMode::Write); + auto file_or_error = Core::File::open(m_destination_path, Core::File::OpenMode::Write); if (file_or_error.is_error()) { GUI::MessageBox::show(window(), DeprecatedString::formatted("Cannot open {} for writing", m_destination_path), "Download failed"sv, GUI::MessageBox::Type::Error); window()->close(); diff --git a/Userland/Applications/Browser/DownloadWidget.h b/Userland/Applications/Browser/DownloadWidget.h index 176d818d8786c9..f60fc9410837c9 100644 --- a/Userland/Applications/Browser/DownloadWidget.h +++ b/Userland/Applications/Browser/DownloadWidget.h @@ -38,7 +38,7 @@ class DownloadWidget final : public GUI::Widget { RefPtr m_close_button; RefPtr m_close_on_finish_checkbox; RefPtr m_browser_image; - OwnPtr m_output_file_stream; + OwnPtr m_output_file_stream; Core::ElapsedTimer m_elapsed_timer; }; diff --git a/Userland/Applications/Browser/main.cpp b/Userland/Applications/Browser/main.cpp index b094e44e0f5f46..049be3a54bc444 100644 --- a/Userland/Applications/Browser/main.cpp +++ b/Userland/Applications/Browser/main.cpp @@ -45,8 +45,8 @@ DeprecatedString g_webdriver_content_ipc_path; static ErrorOr load_content_filters() { - auto file = TRY(Core::Stream::File::open(DeprecatedString::formatted("{}/BrowserContentFilters.txt", Core::StandardPaths::config_directory()), Core::Stream::OpenMode::Read)); - auto ad_filter_list = TRY(Core::Stream::BufferedFile::create(move(file))); + auto file = TRY(Core::File::open(DeprecatedString::formatted("{}/BrowserContentFilters.txt", Core::StandardPaths::config_directory()), Core::File::OpenMode::Read)); + auto ad_filter_list = TRY(Core::BufferedFile::create(move(file))); auto buffer = TRY(ByteBuffer::create_uninitialized(4096)); while (TRY(ad_filter_list->can_read_line())) { auto line = TRY(ad_filter_list->read_line(buffer)); diff --git a/Userland/Applications/BrowserSettings/ContentFilterSettingsWidget.cpp b/Userland/Applications/BrowserSettings/ContentFilterSettingsWidget.cpp index da568273faaa9d..17f44bf3a90fbe 100644 --- a/Userland/Applications/BrowserSettings/ContentFilterSettingsWidget.cpp +++ b/Userland/Applications/BrowserSettings/ContentFilterSettingsWidget.cpp @@ -28,8 +28,8 @@ static DeprecatedString filter_list_file_path() ErrorOr DomainListModel::load() { // FIXME: This should be somewhat shared with Browser. - auto file = TRY(Core::Stream::File::open(filter_list_file_path(), Core::Stream::OpenMode::Read)); - auto content_filter_list = TRY(Core::Stream::BufferedFile::create(move(file))); + auto file = TRY(Core::File::open(filter_list_file_path(), Core::File::OpenMode::Read)); + auto content_filter_list = TRY(Core::BufferedFile::create(move(file))); auto buffer = TRY(ByteBuffer::create_uninitialized(4096)); while (TRY(content_filter_list->can_read_line())) { auto line = TRY(content_filter_list->read_line(buffer)); @@ -50,7 +50,7 @@ ErrorOr DomainListModel::save() for (auto const& domain : m_domain_list) TRY(builder.try_appendff("{}\n", domain)); - auto file = TRY(Core::Stream::File::open(filter_list_file_path(), Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open(filter_list_file_path(), Core::File::OpenMode::Write)); TRY(file->write(builder.to_byte_buffer().bytes())); return {}; } diff --git a/Userland/Applications/DisplaySettings/ThemePreviewWidget.cpp b/Userland/Applications/DisplaySettings/ThemePreviewWidget.cpp index 35abe94a035432..7969cd75541701 100644 --- a/Userland/Applications/DisplaySettings/ThemePreviewWidget.cpp +++ b/Userland/Applications/DisplaySettings/ThemePreviewWidget.cpp @@ -20,7 +20,7 @@ ThemePreviewWidget::ThemePreviewWidget(Gfx::Palette const& palette) ErrorOr ThemePreviewWidget::set_theme(DeprecatedString path) { - auto config_file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto config_file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); TRY(set_theme_from_file(path, move(config_file))); return {}; } diff --git a/Userland/Applications/FileManager/FileOperationProgressWidget.cpp b/Userland/Applications/FileManager/FileOperationProgressWidget.cpp index a185c4e58e51b3..90613a0be0d239 100644 --- a/Userland/Applications/FileManager/FileOperationProgressWidget.cpp +++ b/Userland/Applications/FileManager/FileOperationProgressWidget.cpp @@ -18,7 +18,7 @@ namespace FileManager { -FileOperationProgressWidget::FileOperationProgressWidget(FileOperation operation, NonnullOwnPtr helper_pipe, int helper_pipe_fd) +FileOperationProgressWidget::FileOperationProgressWidget(FileOperation operation, NonnullOwnPtr helper_pipe, int helper_pipe_fd) : m_operation(operation) , m_helper_pipe(move(helper_pipe)) { diff --git a/Userland/Applications/FileManager/FileOperationProgressWidget.h b/Userland/Applications/FileManager/FileOperationProgressWidget.h index 1fc7366f5e45f2..907edd12f568cd 100644 --- a/Userland/Applications/FileManager/FileOperationProgressWidget.h +++ b/Userland/Applications/FileManager/FileOperationProgressWidget.h @@ -8,6 +8,7 @@ #include "FileUtils.h" #include +#include #include #include @@ -21,7 +22,7 @@ class FileOperationProgressWidget : public GUI::Widget { private: // FIXME: The helper_pipe_fd parameter is only needed because we can't get the fd from a Core::Stream. - FileOperationProgressWidget(FileOperation, NonnullOwnPtr helper_pipe, int helper_pipe_fd); + FileOperationProgressWidget(FileOperation, NonnullOwnPtr helper_pipe, int helper_pipe_fd); void did_finish(); void did_error(StringView message); @@ -34,6 +35,6 @@ class FileOperationProgressWidget : public GUI::Widget { FileOperation m_operation; RefPtr m_notifier; - OwnPtr m_helper_pipe; + OwnPtr m_helper_pipe; }; } diff --git a/Userland/Applications/FileManager/FileUtils.cpp b/Userland/Applications/FileManager/FileUtils.cpp index c25b0457a8453e..0f5c3c423fe197 100644 --- a/Userland/Applications/FileManager/FileUtils.cpp +++ b/Userland/Applications/FileManager/FileUtils.cpp @@ -99,8 +99,8 @@ ErrorOr run_file_operation(FileOperation operation, Vectorset_main_widget(operation, move(buffered_pipe), pipe_fds[0])); window->resize(320, 190); diff --git a/Userland/Applications/HexEditor/HexDocument.cpp b/Userland/Applications/HexEditor/HexDocument.cpp index e7e49c6c19cc97..064ca93f91e203 100644 --- a/Userland/Applications/HexEditor/HexDocument.cpp +++ b/Userland/Applications/HexEditor/HexDocument.cpp @@ -5,6 +5,7 @@ */ #include "HexDocument.h" +#include #include void HexDocument::set(size_t position, u8 value) @@ -58,7 +59,7 @@ void HexDocumentMemory::clear_changes() m_changes.clear(); } -ErrorOr HexDocumentMemory::write_to_file(Core::Stream::File& file) +ErrorOr HexDocumentMemory::write_to_file(Core::File& file) { TRY(file.seek(0, SeekMode::SetPosition)); TRY(file.write(m_buffer)); @@ -69,7 +70,7 @@ ErrorOr HexDocumentMemory::write_to_file(Core::Stream::File& file) return {}; } -ErrorOr> HexDocumentFile::create(NonnullOwnPtr file) +ErrorOr> HexDocumentFile::create(NonnullOwnPtr file) { auto document = TRY(adopt_nonnull_own_or_enomem(new HexDocumentFile(move(file)))); TRY(document->initialize_internal_state()); @@ -77,7 +78,7 @@ ErrorOr> HexDocumentFile::create(NonnullOwnPtr file) +HexDocumentFile::HexDocumentFile(NonnullOwnPtr file) : m_file(move(file)) { } @@ -94,7 +95,7 @@ ErrorOr HexDocumentFile::write_to_file() return {}; } -ErrorOr HexDocumentFile::write_to_file(Core::Stream::File& file) +ErrorOr HexDocumentFile::write_to_file(Core::File& file) { TRY(file.truncate(size())); @@ -149,7 +150,7 @@ void HexDocumentFile::clear_changes() m_changes.clear(); } -ErrorOr HexDocumentFile::set_file(NonnullOwnPtr file) +ErrorOr HexDocumentFile::set_file(NonnullOwnPtr file) { m_file = move(file); TRY(initialize_internal_state()); @@ -171,7 +172,7 @@ ErrorOr HexDocumentFile::initialize_internal_state() return {}; } -NonnullOwnPtr const& HexDocumentFile::file() const +NonnullOwnPtr const& HexDocumentFile::file() const { return m_file; } diff --git a/Userland/Applications/HexEditor/HexDocument.h b/Userland/Applications/HexEditor/HexDocument.h index d5e46a4f87771b..256f753f65d860 100644 --- a/Userland/Applications/HexEditor/HexDocument.h +++ b/Userland/Applications/HexEditor/HexDocument.h @@ -52,7 +52,7 @@ class HexDocumentMemory final : public HexDocument { size_t size() const override; Type type() const override; void clear_changes() override; - ErrorOr write_to_file(Core::Stream::File& file); + ErrorOr write_to_file(Core::File& file); private: ByteBuffer m_buffer; @@ -60,16 +60,16 @@ class HexDocumentMemory final : public HexDocument { class HexDocumentFile final : public HexDocument { public: - static ErrorOr> create(NonnullOwnPtr file); + static ErrorOr> create(NonnullOwnPtr file); virtual ~HexDocumentFile() = default; HexDocumentFile(HexDocumentFile&&) = default; HexDocumentFile(HexDocumentFile const&) = delete; - ErrorOr set_file(NonnullOwnPtr file); - NonnullOwnPtr const& file() const; + ErrorOr set_file(NonnullOwnPtr file); + NonnullOwnPtr const& file() const; ErrorOr write_to_file(); - ErrorOr write_to_file(Core::Stream::File& file); + ErrorOr write_to_file(Core::File& file); Cell get(size_t position) override; u8 get_unchanged(size_t position) override; size_t size() const override; @@ -77,12 +77,12 @@ class HexDocumentFile final : public HexDocument { void clear_changes() override; private: - explicit HexDocumentFile(NonnullOwnPtr file); + explicit HexDocumentFile(NonnullOwnPtr file); ErrorOr initialize_internal_state(); void ensure_position_in_buffer(size_t position); - NonnullOwnPtr m_file; + NonnullOwnPtr m_file; size_t m_file_size; Array m_buffer; diff --git a/Userland/Applications/HexEditor/HexEditor.cpp b/Userland/Applications/HexEditor/HexEditor.cpp index e3c28811db3090..18922eeb76e5fa 100644 --- a/Userland/Applications/HexEditor/HexEditor.cpp +++ b/Userland/Applications/HexEditor/HexEditor.cpp @@ -63,7 +63,7 @@ ErrorOr HexEditor::open_new_file(size_t size) return {}; } -void HexEditor::open_file(NonnullOwnPtr file) +void HexEditor::open_file(NonnullOwnPtr file) { m_document = HexDocumentFile::create(move(file)).release_value_but_fixme_should_propagate_errors(); set_content_length(m_document->size()); @@ -136,7 +136,7 @@ void HexEditor::set_selection(size_t position, size_t length) update_status(); } -ErrorOr HexEditor::save_as(NonnullOwnPtr new_file) +ErrorOr HexEditor::save_as(NonnullOwnPtr new_file) { if (m_document->type() == HexDocument::Type::File) { auto& file_document = static_cast(*m_document); diff --git a/Userland/Applications/HexEditor/HexEditor.h b/Userland/Applications/HexEditor/HexEditor.h index 4a27d486d5be11..89eeeb3b5b503e 100644 --- a/Userland/Applications/HexEditor/HexEditor.h +++ b/Userland/Applications/HexEditor/HexEditor.h @@ -35,10 +35,10 @@ class HexEditor : public GUI::AbstractScrollableWidget { size_t buffer_size() const { return m_document->size(); } ErrorOr open_new_file(size_t size); - void open_file(NonnullOwnPtr file); + void open_file(NonnullOwnPtr file); ErrorOr fill_selection(u8 fill_byte); Optional get_byte(size_t position); - ErrorOr save_as(NonnullOwnPtr); + ErrorOr save_as(NonnullOwnPtr); ErrorOr save(); bool undo(); diff --git a/Userland/Applications/HexEditor/HexEditorWidget.cpp b/Userland/Applications/HexEditor/HexEditorWidget.cpp index 64a43c3e3efccf..9084e049b9df3c 100644 --- a/Userland/Applications/HexEditor/HexEditorWidget.cpp +++ b/Userland/Applications/HexEditor/HexEditorWidget.cpp @@ -121,7 +121,7 @@ HexEditorWidget::HexEditorWidget() if (!request_close()) return; - auto response = FileSystemAccessClient::Client::the().open_file(window(), {}, Core::StandardPaths::home_directory(), Core::Stream::OpenMode::ReadWrite); + auto response = FileSystemAccessClient::Client::the().open_file(window(), {}, Core::StandardPaths::home_directory(), Core::File::OpenMode::ReadWrite); if (response.is_error()) return; @@ -142,7 +142,7 @@ HexEditorWidget::HexEditorWidget() }); m_save_as_action = GUI::CommonActions::make_save_as_action([&](auto&) { - auto response = FileSystemAccessClient::Client::the().save_file(window(), m_name, m_extension, Core::Stream::OpenMode::ReadWrite | Core::Stream::OpenMode::Truncate); + auto response = FileSystemAccessClient::Client::the().save_file(window(), m_name, m_extension, Core::File::OpenMode::ReadWrite | Core::File::OpenMode::Truncate); if (response.is_error()) return; auto file = response.release_value(); @@ -525,7 +525,7 @@ void HexEditorWidget::update_title() window()->set_title(builder.to_deprecated_string()); } -void HexEditorWidget::open_file(String const& filename, NonnullOwnPtr file) +void HexEditorWidget::open_file(String const& filename, NonnullOwnPtr file) { window()->set_modified(false); m_editor->open_file(move(file)); @@ -586,7 +586,7 @@ void HexEditorWidget::drop_event(GUI::DropEvent& event) return; // TODO: A drop event should be considered user consent for opening a file - auto response = FileSystemAccessClient::Client::the().request_file(window(), urls.first().path(), Core::Stream::OpenMode::Read); + auto response = FileSystemAccessClient::Client::the().request_file(window(), urls.first().path(), Core::File::OpenMode::Read); if (response.is_error()) return; open_file(response.value().filename(), response.value().release_stream()); diff --git a/Userland/Applications/HexEditor/HexEditorWidget.h b/Userland/Applications/HexEditor/HexEditorWidget.h index addddd246fc43e..a919de7f36a0ce 100644 --- a/Userland/Applications/HexEditor/HexEditorWidget.h +++ b/Userland/Applications/HexEditor/HexEditorWidget.h @@ -25,7 +25,7 @@ class HexEditorWidget final : public GUI::Widget { C_OBJECT(HexEditorWidget) public: virtual ~HexEditorWidget() override = default; - void open_file(String const& filename, NonnullOwnPtr); + void open_file(String const& filename, NonnullOwnPtr); void initialize_menubar(GUI::Window&); bool request_close(); diff --git a/Userland/Applications/HexEditor/main.cpp b/Userland/Applications/HexEditor/main.cpp index f3311bc0e60a5b..185653455b3945 100644 --- a/Userland/Applications/HexEditor/main.cpp +++ b/Userland/Applications/HexEditor/main.cpp @@ -53,7 +53,7 @@ ErrorOr serenity_main(Main::Arguments arguments) if (arguments.argc > 1) { // FIXME: Using `try_request_file_read_only_approved` doesn't work here since the file stored in the editor is only readable. - auto response = FileSystemAccessClient::Client::the().request_file(window, arguments.strings[1], Core::Stream::OpenMode::ReadWrite); + auto response = FileSystemAccessClient::Client::the().request_file(window, arguments.strings[1], Core::File::OpenMode::ReadWrite); if (response.is_error()) return 1; hex_editor_widget->open_file(response.value().filename(), response.value().release_stream()); diff --git a/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp b/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp index a602dbaf101292..678fd1a573d41c 100644 --- a/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp +++ b/Userland/Applications/KeyboardMapper/KeyboardMapperWidget.cpp @@ -8,6 +8,7 @@ #include "KeyboardMapperWidget.h" #include "KeyPositions.h" +#include #include #include #include @@ -191,7 +192,7 @@ ErrorOr KeyboardMapperWidget::save_to_file(StringView filename) // Write to file. DeprecatedString file_content = map_json.to_deprecated_string(); - auto file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open(filename, Core::File::OpenMode::Write)); TRY(file->write(file_content.bytes())); file->close(); diff --git a/Userland/Applications/NetworkSettings/NetworkSettingsWidget.cpp b/Userland/Applications/NetworkSettings/NetworkSettingsWidget.cpp index a57685df360e0c..e8698e2d06add0 100644 --- a/Userland/Applications/NetworkSettings/NetworkSettingsWidget.cpp +++ b/Userland/Applications/NetworkSettings/NetworkSettingsWidget.cpp @@ -62,7 +62,7 @@ NetworkSettingsWidget::NetworkSettingsWidget() auto config_file = Core::ConfigFile::open_for_system("Network").release_value_but_fixme_should_propagate_errors(); - auto proc_net_adapters_file = Core::Stream::File::open("/sys/kernel/net/adapters"sv, Core::Stream::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); + auto proc_net_adapters_file = Core::File::open("/sys/kernel/net/adapters"sv, Core::File::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); auto data = proc_net_adapters_file->read_until_eof().release_value_but_fixme_should_propagate_errors(); JsonParser parser(data); JsonValue proc_net_adapters_json = parser.parse().release_value_but_fixme_should_propagate_errors(); diff --git a/Userland/Applications/PDFViewer/PDFViewerWidget.cpp b/Userland/Applications/PDFViewer/PDFViewerWidget.cpp index b98629302e803f..3d0210dee41208 100644 --- a/Userland/Applications/PDFViewer/PDFViewerWidget.cpp +++ b/Userland/Applications/PDFViewer/PDFViewerWidget.cpp @@ -348,7 +348,7 @@ void PDFViewerWidget::initialize_toolbar(GUI::Toolbar& toolbar) m_show_images->on_checked = [&](auto checked) { m_viewer->set_show_images(checked); }; } -void PDFViewerWidget::open_file(StringView path, NonnullOwnPtr file) +void PDFViewerWidget::open_file(StringView path, NonnullOwnPtr file) { auto maybe_error = try_open_file(path, move(file)); if (maybe_error.is_error()) { @@ -359,7 +359,7 @@ void PDFViewerWidget::open_file(StringView path, NonnullOwnPtr PDFViewerWidget::try_open_file(StringView path, NonnullOwnPtr file) +PDF::PDFErrorOr PDFViewerWidget::try_open_file(StringView path, NonnullOwnPtr file) { window()->set_title(DeprecatedString::formatted("{} - PDF Viewer", path)); diff --git a/Userland/Applications/PDFViewer/PDFViewerWidget.h b/Userland/Applications/PDFViewer/PDFViewerWidget.h index cd1171f2593eee..822d55da885110 100644 --- a/Userland/Applications/PDFViewer/PDFViewerWidget.h +++ b/Userland/Applications/PDFViewer/PDFViewerWidget.h @@ -26,13 +26,13 @@ class PDFViewerWidget final : public GUI::Widget { ~PDFViewerWidget() override = default; void initialize_menubar(GUI::Window&); - void open_file(StringView path, NonnullOwnPtr file); + void open_file(StringView path, NonnullOwnPtr file); private: PDFViewerWidget(); void initialize_toolbar(GUI::Toolbar&); - PDF::PDFErrorOr try_open_file(StringView path, NonnullOwnPtr file); + PDF::PDFErrorOr try_open_file(StringView path, NonnullOwnPtr file); RefPtr m_viewer; RefPtr m_sidebar; diff --git a/Userland/Applications/PixelPaint/ImageEditor.cpp b/Userland/Applications/PixelPaint/ImageEditor.cpp index 0e058e85155f05..0d7ce8f33931b7 100644 --- a/Userland/Applications/PixelPaint/ImageEditor.cpp +++ b/Userland/Applications/PixelPaint/ImageEditor.cpp @@ -733,7 +733,7 @@ void ImageEditor::save_project() save_project_as(); return; } - auto response = FileSystemAccessClient::Client::the().request_file(window(), path(), Core::Stream::OpenMode::Truncate | Core::Stream::OpenMode::Write); + auto response = FileSystemAccessClient::Client::the().request_file(window(), path(), Core::File::OpenMode::Truncate | Core::File::OpenMode::Write); if (response.is_error()) return; auto result = save_project_to_file(response.value().release_stream()); @@ -760,7 +760,7 @@ void ImageEditor::save_project_as() set_unmodified(); } -ErrorOr ImageEditor::save_project_to_file(NonnullOwnPtr file) const +ErrorOr ImageEditor::save_project_to_file(NonnullOwnPtr file) const { StringBuilder builder; auto json = TRY(JsonObjectSerializer<>::try_create(builder)); diff --git a/Userland/Applications/PixelPaint/ImageEditor.h b/Userland/Applications/PixelPaint/ImageEditor.h index 727868247f5ee5..9142ca67ee8ad0 100644 --- a/Userland/Applications/PixelPaint/ImageEditor.h +++ b/Userland/Applications/PixelPaint/ImageEditor.h @@ -153,7 +153,7 @@ class ImageEditor final GUI::MouseEvent event_adjusted_for_layer(GUI::MouseEvent const&, Layer const&) const; GUI::MouseEvent event_with_pan_and_scale_applied(GUI::MouseEvent const&) const; - ErrorOr save_project_to_file(NonnullOwnPtr) const; + ErrorOr save_project_to_file(NonnullOwnPtr) const; int calculate_ruler_step_size() const; Gfx::IntRect mouse_indicator_rect_x() const; diff --git a/Userland/Applications/PixelPaint/MainWidget.cpp b/Userland/Applications/PixelPaint/MainWidget.cpp index 8138870e98250e..f54f99f2648639 100644 --- a/Userland/Applications/PixelPaint/MainWidget.cpp +++ b/Userland/Applications/PixelPaint/MainWidget.cpp @@ -1288,7 +1288,7 @@ void MainWidget::drop_event(GUI::DropEvent& event) if (url.scheme() != "file") continue; - auto response = FileSystemAccessClient::Client::the().request_file(window(), url.path(), Core::Stream::OpenMode::Read); + auto response = FileSystemAccessClient::Client::the().request_file(window(), url.path(), Core::File::OpenMode::Read); if (response.is_error()) return; open_image(response.release_value()); diff --git a/Userland/Applications/PixelPaint/PaletteWidget.cpp b/Userland/Applications/PixelPaint/PaletteWidget.cpp index 96fb38965a0786..aab595cb6460f1 100644 --- a/Userland/Applications/PixelPaint/PaletteWidget.cpp +++ b/Userland/Applications/PixelPaint/PaletteWidget.cpp @@ -224,11 +224,11 @@ Vector PaletteWidget::colors() return colors; } -ErrorOr> PaletteWidget::load_palette_file(NonnullOwnPtr file) +ErrorOr> PaletteWidget::load_palette_file(NonnullOwnPtr file) { Vector palette; Array buffer; - auto buffered_file = TRY(Core::Stream::BufferedFile::create(move(file))); + auto buffered_file = TRY(Core::BufferedFile::create(move(file))); while (TRY(buffered_file->can_read_line())) { auto line = TRY(buffered_file->read_line(buffer)); @@ -252,11 +252,11 @@ ErrorOr> PaletteWidget::load_palette_file(NonnullOwnPtr> PaletteWidget::load_palette_path(DeprecatedString const& file_path) { - auto file = TRY(Core::Stream::File::open(file_path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(file_path, Core::File::OpenMode::Read)); return load_palette_file(move(file)); } -ErrorOr PaletteWidget::save_palette_file(Vector palette, NonnullOwnPtr file) +ErrorOr PaletteWidget::save_palette_file(Vector palette, NonnullOwnPtr file) { for (auto& color : palette) { TRY(file->write_entire_buffer(color.to_deprecated_string_without_alpha().bytes())); diff --git a/Userland/Applications/PixelPaint/PaletteWidget.h b/Userland/Applications/PixelPaint/PaletteWidget.h index aedfce34380de6..362c29f5180ba6 100644 --- a/Userland/Applications/PixelPaint/PaletteWidget.h +++ b/Userland/Applications/PixelPaint/PaletteWidget.h @@ -31,9 +31,9 @@ class PaletteWidget final : public GUI::Frame { Vector colors(); - static ErrorOr> load_palette_file(NonnullOwnPtr); + static ErrorOr> load_palette_file(NonnullOwnPtr); static ErrorOr> load_palette_path(DeprecatedString const&); - static ErrorOr save_palette_file(Vector, NonnullOwnPtr); + static ErrorOr save_palette_file(Vector, NonnullOwnPtr); static Vector fallback_colors(); diff --git a/Userland/Applications/PixelPaint/ProjectLoader.cpp b/Userland/Applications/PixelPaint/ProjectLoader.cpp index 020beb5423fd9f..c5ff7619862673 100644 --- a/Userland/Applications/PixelPaint/ProjectLoader.cpp +++ b/Userland/Applications/PixelPaint/ProjectLoader.cpp @@ -15,7 +15,7 @@ namespace PixelPaint { -ErrorOr ProjectLoader::load_from_file(NonnullOwnPtr file) +ErrorOr ProjectLoader::load_from_file(NonnullOwnPtr file) { auto contents = TRY(file->read_until_eof()); diff --git a/Userland/Applications/PixelPaint/ProjectLoader.h b/Userland/Applications/PixelPaint/ProjectLoader.h index d16c7fef1af829..111964e11c241f 100644 --- a/Userland/Applications/PixelPaint/ProjectLoader.h +++ b/Userland/Applications/PixelPaint/ProjectLoader.h @@ -18,7 +18,7 @@ class ProjectLoader { ProjectLoader() = default; ~ProjectLoader() = default; - ErrorOr load_from_file(NonnullOwnPtr); + ErrorOr load_from_file(NonnullOwnPtr); bool is_raw_image() const { return m_is_raw_image; } bool has_image() const { return !m_image.is_null(); } diff --git a/Userland/Applications/Presenter/Presentation.cpp b/Userland/Applications/Presenter/Presentation.cpp index 2959022f10e3a8..a90da1e9db0fc0 100644 --- a/Userland/Applications/Presenter/Presentation.cpp +++ b/Userland/Applications/Presenter/Presentation.cpp @@ -7,6 +7,7 @@ #include "Presentation.h" #include +#include #include #include #include @@ -79,7 +80,7 @@ ErrorOr> Presentation::load_from_file(StringView fil { if (file_name.is_empty()) return ENOENT; - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(file_name, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(file_name, Core::File::OpenMode::Read)); auto contents = TRY(file->read_until_eof()); auto content_string = StringView { contents }; auto json = TRY(JsonValue::from_string(content_string)); diff --git a/Userland/Applications/Run/RunWindow.cpp b/Userland/Applications/Run/RunWindow.cpp index 4c86e9df8eec59..fc82de3cd628fd 100644 --- a/Userland/Applications/Run/RunWindow.cpp +++ b/Userland/Applications/Run/RunWindow.cpp @@ -170,8 +170,8 @@ DeprecatedString RunWindow::history_file_path() ErrorOr RunWindow::load_history() { m_path_history.clear(); - auto file = TRY(Core::Stream::File::open(history_file_path(), Core::Stream::OpenMode::Read)); - auto buffered_file = TRY(Core::Stream::BufferedFile::create(move(file))); + auto file = TRY(Core::File::open(history_file_path(), Core::File::OpenMode::Read)); + auto buffered_file = TRY(Core::BufferedFile::create(move(file))); Array line_buffer; while (!buffered_file->is_eof()) { @@ -184,7 +184,7 @@ ErrorOr RunWindow::load_history() ErrorOr RunWindow::save_history() { - auto file = TRY(Core::Stream::File::open(history_file_path(), Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open(history_file_path(), Core::File::OpenMode::Write)); // Write the first 25 items of history for (int i = 0; i < min(static_cast(m_path_history.size()), 25); i++) diff --git a/Userland/Applications/SoundPlayer/M3UParser.cpp b/Userland/Applications/SoundPlayer/M3UParser.cpp index 5cd76a45ccbe9f..43353b202be0b8 100644 --- a/Userland/Applications/SoundPlayer/M3UParser.cpp +++ b/Userland/Applications/SoundPlayer/M3UParser.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include M3UParser::M3UParser() @@ -17,7 +18,7 @@ M3UParser::M3UParser() NonnullOwnPtr M3UParser::from_file(StringView path) { - auto file_result = Core::Stream::File::open(path, Core::Stream::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); + auto file_result = Core::File::open(path, Core::File::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); auto contents = file_result->read_until_eof().release_value_but_fixme_should_propagate_errors(); auto use_utf8 = path.ends_with(".m3u8"sv, CaseSensitivity::CaseInsensitive); return from_memory(DeprecatedString { contents, NoChomp }, use_utf8); diff --git a/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp b/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp index 052bdae08843c3..bd4f6927ef907f 100644 --- a/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp +++ b/Userland/Applications/SpaceAnalyzer/TreeMapWidget.cpp @@ -375,7 +375,7 @@ void TreeMapWidget::recalculate_path_for_new_tree() static ErrorOr fill_mounts(Vector& output) { // Output info about currently mounted filesystems. - auto file = TRY(Core::Stream::File::open("/sys/kernel/df"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/df"sv, Core::File::OpenMode::Read)); auto content = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(content)); diff --git a/Userland/Applications/Spreadsheet/ExportDialog.cpp b/Userland/Applications/Spreadsheet/ExportDialog.cpp index df225e488818e7..35b1ca676f7cd5 100644 --- a/Userland/Applications/Spreadsheet/ExportDialog.cpp +++ b/Userland/Applications/Spreadsheet/ExportDialog.cpp @@ -178,7 +178,7 @@ void CSVExportDialogPage::update_preview() m_data_preview_text_editor->set_text(DeprecatedString::formatted("Cannot update preview: {}", maybe_error.error())); } -ErrorOr ExportDialog::make_and_run_for(StringView mime, Core::Stream::File& file, DeprecatedString filename, Workbook& workbook) +ErrorOr ExportDialog::make_and_run_for(StringView mime, Core::File& file, DeprecatedString filename, Workbook& workbook) { auto wizard = GUI::WizardDialog::construct(GUI::Application::the()->active_window()); wizard->set_title("File Export Wizard"); diff --git a/Userland/Applications/Spreadsheet/ExportDialog.h b/Userland/Applications/Spreadsheet/ExportDialog.h index 3579e135aa60a0..c3e5fab677a32b 100644 --- a/Userland/Applications/Spreadsheet/ExportDialog.h +++ b/Userland/Applications/Spreadsheet/ExportDialog.h @@ -58,7 +58,7 @@ struct CSVExportDialogPage { }; struct ExportDialog { - static ErrorOr make_and_run_for(StringView mime, Core::Stream::File&, DeprecatedString filename, Workbook&); + static ErrorOr make_and_run_for(StringView mime, Core::File&, DeprecatedString filename, Workbook&); }; } diff --git a/Userland/Applications/Spreadsheet/ImportDialog.cpp b/Userland/Applications/Spreadsheet/ImportDialog.cpp index f27cfdf8b27e78..2da65ae87d062c 100644 --- a/Userland/Applications/Spreadsheet/ImportDialog.cpp +++ b/Userland/Applications/Spreadsheet/ImportDialog.cpp @@ -174,7 +174,7 @@ void CSVImportDialogPage::update_preview() m_data_preview_table_view->update(); } -ErrorOr, DeprecatedString> ImportDialog::make_and_run_for(GUI::Window& parent, StringView mime, String const& filename, Core::Stream::File& file, Workbook& workbook) +ErrorOr, DeprecatedString> ImportDialog::make_and_run_for(GUI::Window& parent, StringView mime, String const& filename, Core::File& file, Workbook& workbook) { auto wizard = GUI::WizardDialog::construct(&parent); wizard->set_title("File Import Wizard"); diff --git a/Userland/Applications/Spreadsheet/ImportDialog.h b/Userland/Applications/Spreadsheet/ImportDialog.h index 7e8b5b00f31c06..dba25312c63cba 100644 --- a/Userland/Applications/Spreadsheet/ImportDialog.h +++ b/Userland/Applications/Spreadsheet/ImportDialog.h @@ -55,7 +55,7 @@ struct CSVImportDialogPage { }; struct ImportDialog { - static ErrorOr, DeprecatedString> make_and_run_for(GUI::Window& parent, StringView mime, String const& filename, Core::Stream::File& file, Workbook&); + static ErrorOr, DeprecatedString> make_and_run_for(GUI::Window& parent, StringView mime, String const& filename, Core::File& file, Workbook&); }; } diff --git a/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp b/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp index d2971866625c53..5d8fcc72fa7142 100644 --- a/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp +++ b/Userland/Applications/Spreadsheet/SpreadsheetWidget.cpp @@ -147,7 +147,7 @@ SpreadsheetWidget::SpreadsheetWidget(GUI::Window& parent_window, NonnullRefPtrVe return; } - auto response = FileSystemAccessClient::Client::the().request_file(window(), current_filename(), Core::Stream::OpenMode::Write); + auto response = FileSystemAccessClient::Client::the().request_file(window(), current_filename(), Core::File::OpenMode::Write); if (response.is_error()) return; save(response.value().filename(), response.value().stream()); @@ -493,7 +493,7 @@ void SpreadsheetWidget::change_cell_static_color_format(Spreadsheet::FormatType } } -void SpreadsheetWidget::save(String const& filename, Core::Stream::File& file) +void SpreadsheetWidget::save(String const& filename, Core::File& file) { auto result = m_workbook->write_to_file(filename, file); if (result.is_error()) { @@ -504,7 +504,7 @@ void SpreadsheetWidget::save(String const& filename, Core::Stream::File& file) window()->set_modified(false); } -void SpreadsheetWidget::load_file(String const& filename, Core::Stream::File& file) +void SpreadsheetWidget::load_file(String const& filename, Core::File& file) { auto result = m_workbook->open_file(filename, file); if (result.is_error()) { @@ -523,7 +523,7 @@ void SpreadsheetWidget::load_file(String const& filename, Core::Stream::File& fi update_window_title(); } -void SpreadsheetWidget::import_sheets(String const& filename, Core::Stream::File& file) +void SpreadsheetWidget::import_sheets(String const& filename, Core::File& file) { auto result = m_workbook->import_file(filename, file); if (result.is_error()) { diff --git a/Userland/Applications/Spreadsheet/SpreadsheetWidget.h b/Userland/Applications/Spreadsheet/SpreadsheetWidget.h index 63fe723b89bfbb..31d21c7f5c055f 100644 --- a/Userland/Applications/Spreadsheet/SpreadsheetWidget.h +++ b/Userland/Applications/Spreadsheet/SpreadsheetWidget.h @@ -24,9 +24,9 @@ class SpreadsheetWidget final public: virtual ~SpreadsheetWidget() override = default; - void save(String const& filename, Core::Stream::File&); - void load_file(String const& filename, Core::Stream::File&); - void import_sheets(String const& filename, Core::Stream::File&); + void save(String const& filename, Core::File&); + void load_file(String const& filename, Core::File&); + void import_sheets(String const& filename, Core::File&); bool request_close(); void add_sheet(); void add_sheet(NonnullRefPtr&&); diff --git a/Userland/Applications/Spreadsheet/Workbook.cpp b/Userland/Applications/Spreadsheet/Workbook.cpp index 16c0ba407c613e..3df8daa5015df6 100644 --- a/Userland/Applications/Spreadsheet/Workbook.cpp +++ b/Userland/Applications/Spreadsheet/Workbook.cpp @@ -50,7 +50,7 @@ bool Workbook::set_filename(DeprecatedString const& filename) return true; } -ErrorOr Workbook::open_file(String const& filename, Core::Stream::File& file) +ErrorOr Workbook::open_file(String const& filename, Core::File& file) { auto mime = Core::guess_mime_type_based_on_filename(filename); @@ -62,7 +62,7 @@ ErrorOr Workbook::open_file(String const& filename, Core return {}; } -ErrorOr Workbook::write_to_file(String const& filename, Core::Stream::File& stream) +ErrorOr Workbook::write_to_file(String const& filename, Core::File& stream) { auto mime = Core::guess_mime_type_based_on_filename(filename); @@ -74,7 +74,7 @@ ErrorOr Workbook::write_to_file(String const& filename, Core::Stream::File return {}; } -ErrorOr Workbook::import_file(String const& filename, Core::Stream::File& file) +ErrorOr Workbook::import_file(String const& filename, Core::File& file) { auto mime = Core::guess_mime_type_based_on_filename(filename); diff --git a/Userland/Applications/Spreadsheet/Workbook.h b/Userland/Applications/Spreadsheet/Workbook.h index 2020a23943e4f5..84d394053deefb 100644 --- a/Userland/Applications/Spreadsheet/Workbook.h +++ b/Userland/Applications/Spreadsheet/Workbook.h @@ -16,10 +16,10 @@ class Workbook { public: Workbook(NonnullRefPtrVector&& sheets, GUI::Window& parent_window); - ErrorOr open_file(String const& filename, Core::Stream::File&); - ErrorOr write_to_file(String const& filename, Core::Stream::File&); + ErrorOr open_file(String const& filename, Core::File&); + ErrorOr write_to_file(String const& filename, Core::File&); - ErrorOr import_file(String const& filename, Core::Stream::File&); + ErrorOr import_file(String const& filename, Core::File&); DeprecatedString const& current_filename() const { return m_current_filename; } bool set_filename(DeprecatedString const& filename); diff --git a/Userland/Applications/SystemMonitor/ProcessModel.cpp b/Userland/Applications/SystemMonitor/ProcessModel.cpp index f32d84ad25b13e..04d031269c5ace 100644 --- a/Userland/Applications/SystemMonitor/ProcessModel.cpp +++ b/Userland/Applications/SystemMonitor/ProcessModel.cpp @@ -410,7 +410,7 @@ Vector ProcessModel::matches(StringView searching, unsigned fla static ErrorOr try_read_command_line(pid_t pid) { - auto file = TRY(Core::Stream::File::open(DeprecatedString::formatted("/proc/{}/cmdline", pid), Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(DeprecatedString::formatted("/proc/{}/cmdline", pid), Core::File::OpenMode::Read)); auto data = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(StringView { data.bytes() })); auto array = json.as_array().values(); diff --git a/Userland/Applications/TextEditor/MainWidget.cpp b/Userland/Applications/TextEditor/MainWidget.cpp index 99a42a31783ddb..a95a47bad620ff 100644 --- a/Userland/Applications/TextEditor/MainWidget.cpp +++ b/Userland/Applications/TextEditor/MainWidget.cpp @@ -303,7 +303,7 @@ MainWidget::MainWidget() m_save_as_action->activate(); return; } - auto response = FileSystemAccessClient::Client::the().request_file(window(), m_path, Core::Stream::OpenMode::Truncate | Core::Stream::OpenMode::Write); + auto response = FileSystemAccessClient::Client::the().request_file(window(), m_path, Core::File::OpenMode::Truncate | Core::File::OpenMode::Write); if (response.is_error()) return; @@ -746,7 +746,7 @@ void MainWidget::update_title() window()->set_title(builder.to_deprecated_string()); } -ErrorOr MainWidget::read_file(String const& filename, Core::Stream::File& file) +ErrorOr MainWidget::read_file(String const& filename, Core::File& file) { m_editor->set_text(TRY(file.read_until_eof())); set_path(filename); diff --git a/Userland/Applications/TextEditor/MainWidget.h b/Userland/Applications/TextEditor/MainWidget.h index 04ff814e2221f9..61c37fefe640f6 100644 --- a/Userland/Applications/TextEditor/MainWidget.h +++ b/Userland/Applications/TextEditor/MainWidget.h @@ -25,7 +25,7 @@ class MainWidget final : public GUI::Widget { public: virtual ~MainWidget() override = default; - ErrorOr read_file(String const& filename, Core::Stream::File&); + ErrorOr read_file(String const& filename, Core::File&); void open_nonexistent_file(DeprecatedString const& path); bool request_close(); diff --git a/Userland/Applications/ThemeEditor/MainWidget.cpp b/Userland/Applications/ThemeEditor/MainWidget.cpp index ff4e6453ddf315..b80dcb5ffe9049 100644 --- a/Userland/Applications/ThemeEditor/MainWidget.cpp +++ b/Userland/Applications/ThemeEditor/MainWidget.cpp @@ -250,12 +250,12 @@ ErrorOr MainWidget::initialize_menubar(GUI::Window& window) m_save_action = GUI::CommonActions::make_save_action([&](auto&) { if (m_path.has_value()) { - auto result = FileSystemAccessClient::Client::the().request_file(&window, *m_path, Core::Stream::OpenMode::ReadWrite | Core::Stream::OpenMode::Truncate); + auto result = FileSystemAccessClient::Client::the().request_file(&window, *m_path, Core::File::OpenMode::ReadWrite | Core::File::OpenMode::Truncate); if (result.is_error()) return; save_to_file(result.value().filename(), result.value().release_stream()); } else { - auto result = FileSystemAccessClient::Client::the().save_file(&window, "Theme", "ini", Core::Stream::OpenMode::ReadWrite | Core::Stream::OpenMode::Truncate); + auto result = FileSystemAccessClient::Client::the().save_file(&window, "Theme", "ini", Core::File::OpenMode::ReadWrite | Core::File::OpenMode::Truncate); if (result.is_error()) return; save_to_file(result.value().filename(), result.value().release_stream()); @@ -264,7 +264,7 @@ ErrorOr MainWidget::initialize_menubar(GUI::Window& window) TRY(file_menu->try_add_action(*m_save_action)); TRY(file_menu->try_add_action(GUI::CommonActions::make_save_as_action([&](auto&) { - auto result = FileSystemAccessClient::Client::the().save_file(&window, "Theme", "ini", Core::Stream::OpenMode::ReadWrite | Core::Stream::OpenMode::Truncate); + auto result = FileSystemAccessClient::Client::the().save_file(&window, "Theme", "ini", Core::File::OpenMode::ReadWrite | Core::File::OpenMode::Truncate); if (result.is_error()) return; save_to_file(result.value().filename(), result.value().release_stream()); @@ -315,7 +315,7 @@ void MainWidget::set_path(DeprecatedString path) update_title(); } -void MainWidget::save_to_file(String const& filename, NonnullOwnPtr file) +void MainWidget::save_to_file(String const& filename, NonnullOwnPtr file) { auto theme = Core::ConfigFile::open(filename.to_deprecated_string(), move(file)).release_value_but_fixme_should_propagate_errors(); @@ -601,7 +601,7 @@ void MainWidget::show_path_picker_dialog(StringView property_display_name, GUI:: path_input.set_text(*result); } -ErrorOr MainWidget::load_from_file(String const& filename, NonnullOwnPtr file) +ErrorOr MainWidget::load_from_file(String const& filename, NonnullOwnPtr file) { auto config_file = TRY(Core::ConfigFile::open(filename.to_deprecated_string(), move(file))); auto theme = TRY(Gfx::load_system_theme(config_file)); @@ -669,7 +669,7 @@ void MainWidget::drop_event(GUI::DropEvent& event) if (request_close() == GUI::Window::CloseRequestDecision::StayOpen) return; - auto response = FileSystemAccessClient::Client::the().request_file(window(), urls.first().path(), Core::Stream::OpenMode::Read); + auto response = FileSystemAccessClient::Client::the().request_file(window(), urls.first().path(), Core::File::OpenMode::Read); if (response.is_error()) return; diff --git a/Userland/Applications/ThemeEditor/MainWidget.h b/Userland/Applications/ThemeEditor/MainWidget.h index d33155de803011..287a165b0485d5 100644 --- a/Userland/Applications/ThemeEditor/MainWidget.h +++ b/Userland/Applications/ThemeEditor/MainWidget.h @@ -86,7 +86,7 @@ class MainWidget final : public GUI::Widget { ErrorOr initialize_menubar(GUI::Window&); GUI::Window::CloseRequestDecision request_close(); void update_title(); - ErrorOr load_from_file(String const& filename, NonnullOwnPtr file); + ErrorOr load_from_file(String const& filename, NonnullOwnPtr file); private: explicit MainWidget(NonnullRefPtr); @@ -94,7 +94,7 @@ class MainWidget final : public GUI::Widget { virtual void drag_enter_event(GUI::DragEvent&) override; virtual void drop_event(GUI::DropEvent&) override; - void save_to_file(String const& filename, NonnullOwnPtr file); + void save_to_file(String const& filename, NonnullOwnPtr file); ErrorOr encode(); void set_path(DeprecatedString); diff --git a/Userland/Applications/Welcome/WelcomeWidget.cpp b/Userland/Applications/Welcome/WelcomeWidget.cpp index 4fef76b0340746..776c37f84bd0e0 100644 --- a/Userland/Applications/Welcome/WelcomeWidget.cpp +++ b/Userland/Applications/Welcome/WelcomeWidget.cpp @@ -95,8 +95,8 @@ ErrorOr WelcomeWidget::create_widgets() ErrorOr WelcomeWidget::open_and_parse_tips_file() { auto path = TRY(String::formatted("{}/tips.txt", Core::StandardPaths::documents_directory())); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); - auto buffered_file = TRY(Core::Stream::BufferedFile::create(move(file))); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); + auto buffered_file = TRY(Core::BufferedFile::create(move(file))); Array buffer; while (TRY(buffered_file->can_read_line())) { diff --git a/Userland/Demos/CatDog/CatDog.cpp b/Userland/Demos/CatDog/CatDog.cpp index c276704ffee5db..70c9dd3cf593c9 100644 --- a/Userland/Demos/CatDog/CatDog.cpp +++ b/Userland/Demos/CatDog/CatDog.cpp @@ -6,6 +6,7 @@ */ #include "CatDog.h" +#include #include #include #include @@ -55,7 +56,7 @@ ErrorOr> CatDog::create() } CatDog::CatDog() - : m_proc_all(MUST(Core::Stream::File::open("/sys/kernel/processes"sv, Core::Stream::OpenMode::Read))) + : m_proc_all(MUST(Core::File::open("/sys/kernel/processes"sv, Core::File::OpenMode::Read))) { m_idle_sleep_timer.start(); } diff --git a/Userland/Demos/CatDog/CatDog.h b/Userland/Demos/CatDog/CatDog.h index f5daca0e4d3851..0a86a013bfa839 100644 --- a/Userland/Demos/CatDog/CatDog.h +++ b/Userland/Demos/CatDog/CatDog.h @@ -74,7 +74,7 @@ class CatDog final : public GUI::Widget Gfx::IntPoint m_mouse_offset {}; Core::ElapsedTimer m_idle_sleep_timer; - NonnullOwnPtr m_proc_all; + NonnullOwnPtr m_proc_all; State m_state { State::Roaming }; State m_frame { State::Frame1 }; diff --git a/Userland/DevTools/GMLPlayground/main.cpp b/Userland/DevTools/GMLPlayground/main.cpp index 4bb2bb2246aa03..f1c9a7d5b0ce4d 100644 --- a/Userland/DevTools/GMLPlayground/main.cpp +++ b/Userland/DevTools/GMLPlayground/main.cpp @@ -157,7 +157,7 @@ ErrorOr serenity_main(Main::Arguments arguments) save_as_action->activate(); return; } - auto response = FileSystemAccessClient::Client::the().request_file(window, file_path, Core::Stream::OpenMode::Truncate | Core::Stream::OpenMode::Write); + auto response = FileSystemAccessClient::Client::the().request_file(window, file_path, Core::File::OpenMode::Truncate | Core::File::OpenMode::Write); if (response.is_error()) return; diff --git a/Userland/DevTools/HackStudio/Editor.cpp b/Userland/DevTools/HackStudio/Editor.cpp index 4fe98a0b61e98c..83bb54c955ed88 100644 --- a/Userland/DevTools/HackStudio/Editor.cpp +++ b/Userland/DevTools/HackStudio/Editor.cpp @@ -232,7 +232,7 @@ void Editor::show_documentation_tooltip_if_available(DeprecatedString const& hov } dbgln_if(EDITOR_DEBUG, "opening {}", it->value); - auto file_or_error = Core::Stream::File::open(it->value, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(it->value, Core::File::OpenMode::Read); if (file_or_error.is_error()) { dbgln("Failed to open {}, {}", it->value, file_or_error.error()); return; diff --git a/Userland/DevTools/HackStudio/Git/GitWidget.cpp b/Userland/DevTools/HackStudio/Git/GitWidget.cpp index e079a84dd5176d..2afe716263331c 100644 --- a/Userland/DevTools/HackStudio/Git/GitWidget.cpp +++ b/Userland/DevTools/HackStudio/Git/GitWidget.cpp @@ -155,7 +155,7 @@ void GitWidget::set_view_diff_callback(ViewDiffCallback callback) void GitWidget::show_diff(DeprecatedString const& file_path) { if (!m_git_repo->is_tracked(file_path)) { - auto file = Core::Stream::File::open(file_path, Core::Stream::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); + auto file = Core::File::open(file_path, Core::File::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); auto content = file->read_until_eof().release_value_but_fixme_should_propagate_errors(); m_view_diff_callback("", Diff::generate_only_additions(content)); return; diff --git a/Userland/DevTools/HackStudio/HackStudioWidget.cpp b/Userland/DevTools/HackStudio/HackStudioWidget.cpp index 464baebc50acd0..deeda7935b353d 100644 --- a/Userland/DevTools/HackStudio/HackStudioWidget.cpp +++ b/Userland/DevTools/HackStudio/HackStudioWidget.cpp @@ -550,7 +550,7 @@ ErrorOr> HackStudioWidget::create_new_file_action(Dep filepath = DeprecatedString::formatted("{}{}", filepath, filename); - auto file_or_error = Core::Stream::File::open(filepath, Core::Stream::OpenMode::Write | Core::Stream::OpenMode::MustBeNew); + auto file_or_error = Core::File::open(filepath, Core::File::OpenMode::Write | Core::File::OpenMode::MustBeNew); if (file_or_error.is_error()) { GUI::MessageBox::show_error(window(), DeprecatedString::formatted("Failed to create '{}': {}", filepath, file_or_error.error())); return; @@ -1791,7 +1791,7 @@ ErrorOr> HackStudioWidget::create_open_project_config if (maybe_error.is_error() && maybe_error.error().code() != EEXIST) return maybe_error.release_error(); - auto file = TRY(Core::Stream::File::open(absolute_config_file_path, Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open(absolute_config_file_path, Core::File::OpenMode::Write)); TRY(file->write_entire_buffer( "{\n" " \"build_command\": \"your build command here\",\n" diff --git a/Userland/DevTools/HackStudio/LanguageServers/FileDB.cpp b/Userland/DevTools/HackStudio/LanguageServers/FileDB.cpp index 27c2091c2bceb3..15673d56a5c7ff 100644 --- a/Userland/DevTools/HackStudio/LanguageServers/FileDB.cpp +++ b/Userland/DevTools/HackStudio/LanguageServers/FileDB.cpp @@ -9,6 +9,7 @@ #include #include #include +#include namespace LanguageServers { @@ -74,13 +75,13 @@ DeprecatedString FileDB::to_absolute_path(DeprecatedString const& filename) cons ErrorOr> FileDB::create_from_filesystem(DeprecatedString const& filename) const { - auto file = TRY(Core::Stream::File::open(to_absolute_path(filename), Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(to_absolute_path(filename), Core::File::OpenMode::Read)); return create_from_file(move(file)); } ErrorOr> FileDB::create_from_fd(int fd) const { - auto file = TRY(Core::Stream::File::adopt_fd(fd, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::adopt_fd(fd, Core::File::OpenMode::Read)); return create_from_file(move(file)); } @@ -101,7 +102,7 @@ class DefaultDocumentClient final : public GUI::TextDocument::Client { }; static DefaultDocumentClient s_default_document_client; -ErrorOr> FileDB::create_from_file(NonnullOwnPtr file) const +ErrorOr> FileDB::create_from_file(NonnullOwnPtr file) const { auto content = TRY(file->read_until_eof()); auto document = GUI::TextDocument::create(&s_default_document_client); diff --git a/Userland/DevTools/HackStudio/LanguageServers/FileDB.h b/Userland/DevTools/HackStudio/LanguageServers/FileDB.h index 86cbaf43572ed9..8cb3302c64886b 100644 --- a/Userland/DevTools/HackStudio/LanguageServers/FileDB.h +++ b/Userland/DevTools/HackStudio/LanguageServers/FileDB.h @@ -34,7 +34,7 @@ class FileDB final : public CodeComprehension::FileDB { private: ErrorOr> create_from_filesystem(DeprecatedString const& filename) const; ErrorOr> create_from_fd(int fd) const; - ErrorOr> create_from_file(NonnullOwnPtr) const; + ErrorOr> create_from_file(NonnullOwnPtr) const; static RefPtr create_with_content(DeprecatedString const&); private: diff --git a/Userland/DevTools/HackStudio/ProjectBuilder.cpp b/Userland/DevTools/HackStudio/ProjectBuilder.cpp index 8fc1c0581225c8..5c714c975347b4 100644 --- a/Userland/DevTools/HackStudio/ProjectBuilder.cpp +++ b/Userland/DevTools/HackStudio/ProjectBuilder.cpp @@ -112,7 +112,7 @@ ErrorOr ProjectBuilder::build_serenity_component() ErrorOr ProjectBuilder::component_name(StringView cmake_file_path) { - auto file = TRY(Core::Stream::File::open(cmake_file_path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(cmake_file_path, Core::File::OpenMode::Read)); auto content = TRY(file->read_until_eof()); static Regex const component_name(R"~~~(serenity_component\([\s]*(\w+)[\s\S]*\))~~~"); @@ -135,7 +135,7 @@ ErrorOr ProjectBuilder::initialize_build_directory() if (Core::DeprecatedFile::exists(cmake_file_path)) MUST(Core::DeprecatedFile::remove(cmake_file_path, Core::DeprecatedFile::RecursionMode::Disallowed)); - auto cmake_file = TRY(Core::Stream::File::open(cmake_file_path, Core::Stream::OpenMode::Write)); + auto cmake_file = TRY(Core::File::open(cmake_file_path, Core::File::OpenMode::Write)); TRY(cmake_file->write_entire_buffer(generate_cmake_file_content().bytes())); TRY(m_terminal->run_command(DeprecatedString::formatted("cmake -S {} -DHACKSTUDIO_BUILD=ON -DHACKSTUDIO_BUILD_CMAKE_FILE={}" diff --git a/Userland/DevTools/HackStudio/ProjectConfig.cpp b/Userland/DevTools/HackStudio/ProjectConfig.cpp index 2171c03b21081c..5eacf415aed493 100644 --- a/Userland/DevTools/HackStudio/ProjectConfig.cpp +++ b/Userland/DevTools/HackStudio/ProjectConfig.cpp @@ -6,6 +6,7 @@ #include "ProjectConfig.h" #include +#include #include namespace HackStudio { @@ -17,7 +18,7 @@ ProjectConfig::ProjectConfig(JsonObject config) ErrorOr> ProjectConfig::try_load_project_config(DeprecatedString path) { - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); diff --git a/Userland/DevTools/HackStudio/ProjectFile.cpp b/Userland/DevTools/HackStudio/ProjectFile.cpp index a7c5129347c039..768085b08c7320 100644 --- a/Userland/DevTools/HackStudio/ProjectFile.cpp +++ b/Userland/DevTools/HackStudio/ProjectFile.cpp @@ -5,6 +5,7 @@ */ #include "ProjectFile.h" +#include #include namespace HackStudio { @@ -54,7 +55,7 @@ void ProjectFile::create_document_if_needed() const return; m_document = CodeDocument::create(m_name); - auto file_or_error = Core::Stream::File::open(m_name, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(m_name, Core::File::OpenMode::Read); if (file_or_error.is_error()) { warnln("Couldn't open '{}': {}", m_name, file_or_error.error()); // This is okay though, we'll just go with an empty document and create the file when saving. diff --git a/Userland/DevTools/Profiler/Profile.cpp b/Userland/DevTools/Profiler/Profile.cpp index 2247fbf1b76855..9d512f029ca955 100644 --- a/Userland/DevTools/Profiler/Profile.cpp +++ b/Userland/DevTools/Profiler/Profile.cpp @@ -236,7 +236,7 @@ OwnPtr g_kernel_debug_info; ErrorOr> Profile::load_from_perfcore_file(StringView path) { - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto json = JsonValue::from_string(TRY(file->read_until_eof())); if (json.is_error() || !json.value().is_object()) diff --git a/Userland/DevTools/Profiler/SourceModel.cpp b/Userland/DevTools/Profiler/SourceModel.cpp index ca478adc240a7e..87d60dc37379dc 100644 --- a/Userland/DevTools/Profiler/SourceModel.cpp +++ b/Userland/DevTools/Profiler/SourceModel.cpp @@ -29,8 +29,8 @@ class SourceFile final { DeprecatedString source_file_name = filename.replace("../../"sv, source_root_path, ReplaceMode::FirstOnly); auto try_read_lines = [&]() -> ErrorOr { - auto unbuffered_file = TRY(Core::Stream::File::open(source_file_name, Core::Stream::OpenMode::Read)); - auto file = TRY(Core::Stream::BufferedFile::create(move(unbuffered_file))); + auto unbuffered_file = TRY(Core::File::open(source_file_name, Core::File::OpenMode::Read)); + auto file = TRY(Core::BufferedFile::create(move(unbuffered_file))); Array buffer; while (!file->is_eof()) diff --git a/Userland/DevTools/SQLStudio/ScriptEditor.cpp b/Userland/DevTools/SQLStudio/ScriptEditor.cpp index 0ad98073016fe2..3c8261b9016e1e 100644 --- a/Userland/DevTools/SQLStudio/ScriptEditor.cpp +++ b/Userland/DevTools/SQLStudio/ScriptEditor.cpp @@ -29,7 +29,7 @@ void ScriptEditor::new_script_with_temp_name(DeprecatedString name) ErrorOr ScriptEditor::open_script_from_file(LexicalPath const& file_path) { - auto file = TRY(Core::Stream::File::open(file_path.string(), Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(file_path.string(), Core::File::OpenMode::Read)); auto buffer = TRY(file->read_until_eof()); set_text({ buffer.bytes() }); @@ -40,7 +40,7 @@ ErrorOr ScriptEditor::open_script_from_file(LexicalPath const& file_path) static ErrorOr save_text_to_file(StringView filename, DeprecatedString text) { - auto file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open(filename, Core::File::OpenMode::Write)); if (!text.is_empty()) TRY(file->write_entire_buffer(text.bytes())); diff --git a/Userland/DevTools/UserspaceEmulator/main.cpp b/Userland/DevTools/UserspaceEmulator/main.cpp index eda37b6d7e2d6f..52e497f69d1b9f 100644 --- a/Userland/DevTools/UserspaceEmulator/main.cpp +++ b/Userland/DevTools/UserspaceEmulator/main.cpp @@ -61,7 +61,7 @@ int main(int argc, char** argv, char** env) OwnPtr> profile_string_id_map; if (dump_profile) { - auto profile_stream_or_error = Core::Stream::File::open(profile_dump_path, Core::Stream::OpenMode::Write); + auto profile_stream_or_error = Core::File::open(profile_dump_path, Core::File::OpenMode::Write); if (profile_stream_or_error.is_error()) { warnln("Failed to open '{}' for writing: {}", profile_dump_path, profile_stream_or_error.error()); return 1; diff --git a/Userland/Games/Chess/ChessWidget.cpp b/Userland/Games/Chess/ChessWidget.cpp index 4c34b056b82b7a..65e0006a608aa4 100644 --- a/Userland/Games/Chess/ChessWidget.cpp +++ b/Userland/Games/Chess/ChessWidget.cpp @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -532,7 +533,7 @@ DeprecatedString ChessWidget::get_fen() const return m_playback ? m_board_playback.to_fen() : m_board.to_fen(); } -ErrorOr ChessWidget::import_pgn(Core::Stream::File& file) +ErrorOr ChessWidget::import_pgn(Core::File& file) { m_board = Chess::Board(); @@ -629,7 +630,7 @@ ErrorOr ChessWidget::import_pgn(Core::Stream::File& file) return {}; } -ErrorOr ChessWidget::export_pgn(Core::Stream::File& file) const +ErrorOr ChessWidget::export_pgn(Core::File& file) const { // Tag Pair Section TRY(file.write("[Event \"Casual Game\"]\n"sv.bytes())); diff --git a/Userland/Games/Chess/ChessWidget.h b/Userland/Games/Chess/ChessWidget.h index ef048ec7ae7d53..3e35f2706fb607 100644 --- a/Userland/Games/Chess/ChessWidget.h +++ b/Userland/Games/Chess/ChessWidget.h @@ -54,8 +54,8 @@ class ChessWidget final void set_show_available_moves(bool e) { m_show_available_moves = e; } DeprecatedString get_fen() const; - ErrorOr import_pgn(Core::Stream::File&); - ErrorOr export_pgn(Core::Stream::File&) const; + ErrorOr import_pgn(Core::File&); + ErrorOr export_pgn(Core::File&) const; int resign(); void flip_board(); diff --git a/Userland/Games/MasterWord/WordGame.cpp b/Userland/Games/MasterWord/WordGame.cpp index a7665458a2bb8b..9a3d1ddfed8bd6 100644 --- a/Userland/Games/MasterWord/WordGame.cpp +++ b/Userland/Games/MasterWord/WordGame.cpp @@ -177,8 +177,8 @@ void WordGame::read_words() m_words.clear(); auto try_load_words = [&]() -> ErrorOr { - auto response = TRY(Core::Stream::File::open("/res/words.txt"sv, Core::Stream::OpenMode::Read)); - auto words_file = TRY(Core::Stream::BufferedFile::create(move(response))); + auto response = TRY(Core::File::open("/res/words.txt"sv, Core::File::OpenMode::Read)); + auto words_file = TRY(Core::BufferedFile::create(move(response))); Array buffer; while (!words_file->is_eof()) { diff --git a/Userland/Libraries/LibAudio/FlacLoader.cpp b/Userland/Libraries/LibAudio/FlacLoader.cpp index 26036610d4c895..2eaa331ed77723 100644 --- a/Userland/Libraries/LibAudio/FlacLoader.cpp +++ b/Userland/Libraries/LibAudio/FlacLoader.cpp @@ -21,6 +21,7 @@ #include #include #include +#include #include namespace Audio { @@ -32,7 +33,7 @@ FlacLoaderPlugin::FlacLoaderPlugin(NonnullOwnPtr stream) Result, LoaderError> FlacLoaderPlugin::create(StringView path) { - auto stream = LOADER_TRY(Core::Stream::BufferedFile::create(LOADER_TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)))); + auto stream = LOADER_TRY(Core::BufferedFile::create(LOADER_TRY(Core::File::open(path, Core::File::OpenMode::Read)))); auto loader = make(move(stream)); LOADER_TRY(loader->initialize()); diff --git a/Userland/Libraries/LibAudio/MP3Loader.cpp b/Userland/Libraries/LibAudio/MP3Loader.cpp index c36287112f3239..0bb9a5873a5701 100644 --- a/Userland/Libraries/LibAudio/MP3Loader.cpp +++ b/Userland/Libraries/LibAudio/MP3Loader.cpp @@ -8,6 +8,7 @@ #include "MP3HuffmanTables.h" #include "MP3Tables.h" #include +#include namespace Audio { @@ -21,7 +22,7 @@ MP3LoaderPlugin::MP3LoaderPlugin(NonnullOwnPtr stream) Result, LoaderError> MP3LoaderPlugin::create(StringView path) { - auto stream = LOADER_TRY(Core::Stream::BufferedFile::create(LOADER_TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)))); + auto stream = LOADER_TRY(Core::BufferedFile::create(LOADER_TRY(Core::File::open(path, Core::File::OpenMode::Read)))); auto loader = make(move(stream)); LOADER_TRY(loader->initialize()); diff --git a/Userland/Libraries/LibAudio/WavLoader.cpp b/Userland/Libraries/LibAudio/WavLoader.cpp index f222cd4cfc9ed1..3a9723883bf389 100644 --- a/Userland/Libraries/LibAudio/WavLoader.cpp +++ b/Userland/Libraries/LibAudio/WavLoader.cpp @@ -13,6 +13,7 @@ #include #include #include +#include namespace Audio { @@ -25,7 +26,7 @@ WavLoaderPlugin::WavLoaderPlugin(NonnullOwnPtr stream) Result, LoaderError> WavLoaderPlugin::create(StringView path) { - auto stream = LOADER_TRY(Core::Stream::BufferedFile::create(LOADER_TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)))); + auto stream = LOADER_TRY(Core::BufferedFile::create(LOADER_TRY(Core::File::open(path, Core::File::OpenMode::Read)))); auto loader = make(move(stream)); LOADER_TRY(loader->initialize()); diff --git a/Userland/Libraries/LibCore/CMakeLists.txt b/Userland/Libraries/LibCore/CMakeLists.txt index 1c0e3577f557d5..f75c9a206804c4 100644 --- a/Userland/Libraries/LibCore/CMakeLists.txt +++ b/Userland/Libraries/LibCore/CMakeLists.txt @@ -10,6 +10,7 @@ set(SOURCES ElapsedTimer.cpp Event.cpp EventLoop.cpp + File.cpp IODevice.cpp LockFile.cpp MappedFile.cpp @@ -25,7 +26,6 @@ set(SOURCES Socket.cpp SOCKSProxyClient.cpp StandardPaths.cpp - Stream.cpp System.cpp SystemServerTakeover.cpp TCPServer.cpp diff --git a/Userland/Libraries/LibCore/ConfigFile.cpp b/Userland/Libraries/LibCore/ConfigFile.cpp index f26f0a072206bd..7da3e9bc9e055e 100644 --- a/Userland/Libraries/LibCore/ConfigFile.cpp +++ b/Userland/Libraries/LibCore/ConfigFile.cpp @@ -40,8 +40,8 @@ ErrorOr> ConfigFile::open_for_system(DeprecatedString ErrorOr> ConfigFile::open(DeprecatedString const& filename, AllowWriting allow_altering) { - auto maybe_file = Stream::File::open(filename, allow_altering == AllowWriting::Yes ? Stream::OpenMode::ReadWrite : Stream::OpenMode::Read); - OwnPtr buffered_file; + auto maybe_file = File::open(filename, allow_altering == AllowWriting::Yes ? File::OpenMode::ReadWrite : File::OpenMode::Read); + OwnPtr buffered_file; if (maybe_file.is_error()) { // If we attempted to open a read-only file that does not exist, we ignore the error, making it appear // the same as if we had opened an empty file. This behavior is a little weird, but is required by @@ -49,7 +49,7 @@ ErrorOr> ConfigFile::open(DeprecatedString const& file if (!(allow_altering == AllowWriting::No && maybe_file.error().code() == ENOENT)) return maybe_file.release_error(); } else { - buffered_file = TRY(Stream::BufferedFile::create(maybe_file.release_value())); + buffered_file = TRY(BufferedFile::create(maybe_file.release_value())); } auto config_file = TRY(adopt_nonnull_ref_or_enomem(new (nothrow) ConfigFile(filename, move(buffered_file)))); @@ -59,20 +59,20 @@ ErrorOr> ConfigFile::open(DeprecatedString const& file ErrorOr> ConfigFile::open(DeprecatedString const& filename, int fd) { - auto file = TRY(Stream::File::adopt_fd(fd, Stream::OpenMode::ReadWrite)); + auto file = TRY(File::adopt_fd(fd, File::OpenMode::ReadWrite)); return open(filename, move(file)); } -ErrorOr> ConfigFile::open(DeprecatedString const& filename, NonnullOwnPtr file) +ErrorOr> ConfigFile::open(DeprecatedString const& filename, NonnullOwnPtr file) { - auto buffered_file = TRY(Stream::BufferedFile::create(move(file))); + auto buffered_file = TRY(BufferedFile::create(move(file))); auto config_file = TRY(adopt_nonnull_ref_or_enomem(new (nothrow) ConfigFile(filename, move(buffered_file)))); TRY(config_file->reparse()); return config_file; } -ConfigFile::ConfigFile(DeprecatedString const& filename, OwnPtr open_file) +ConfigFile::ConfigFile(DeprecatedString const& filename, OwnPtr open_file) : m_filename(filename) , m_file(move(open_file)) { diff --git a/Userland/Libraries/LibCore/ConfigFile.h b/Userland/Libraries/LibCore/ConfigFile.h index 04a81790a29249..29bfc99e2e6828 100644 --- a/Userland/Libraries/LibCore/ConfigFile.h +++ b/Userland/Libraries/LibCore/ConfigFile.h @@ -14,6 +14,7 @@ #include #include #include +#include #include namespace Core { @@ -30,7 +31,7 @@ class ConfigFile : public RefCounted { static ErrorOr> open_for_system(DeprecatedString const& app_name, AllowWriting = AllowWriting::No); static ErrorOr> open(DeprecatedString const& filename, AllowWriting = AllowWriting::No); static ErrorOr> open(DeprecatedString const& filename, int fd); - static ErrorOr> open(DeprecatedString const& filename, NonnullOwnPtr); + static ErrorOr> open(DeprecatedString const& filename, NonnullOwnPtr); ~ConfigFile(); bool has_group(DeprecatedString const&) const; @@ -78,12 +79,12 @@ class ConfigFile : public RefCounted { DeprecatedString const& filename() const { return m_filename; } private: - ConfigFile(DeprecatedString const& filename, OwnPtr open_file); + ConfigFile(DeprecatedString const& filename, OwnPtr open_file); ErrorOr reparse(); DeprecatedString m_filename; - OwnPtr m_file; + OwnPtr m_file; HashMap> m_groups; bool m_dirty { false }; }; diff --git a/Userland/Libraries/LibCore/DeprecatedFile.h b/Userland/Libraries/LibCore/DeprecatedFile.h index 7f9c1b70829131..64c1b19fe98852 100644 --- a/Userland/Libraries/LibCore/DeprecatedFile.h +++ b/Userland/Libraries/LibCore/DeprecatedFile.h @@ -19,7 +19,7 @@ namespace Core { /// /// Use of Core::File for reading/writing data is deprecated. -/// Please use Core::Stream::File and Core::Stream::BufferedFile instead. +/// Please use Core::File and Core::BufferedFile instead. /// class DeprecatedFile final : public IODevice { C_OBJECT(DeprecatedFile) diff --git a/Userland/Libraries/LibCore/Directory.cpp b/Userland/Libraries/LibCore/Directory.cpp index 6881bb60b402bf..5e821d16fc903a 100644 --- a/Userland/Libraries/LibCore/Directory.cpp +++ b/Userland/Libraries/LibCore/Directory.cpp @@ -89,10 +89,10 @@ ErrorOr Directory::path() const return m_path.value(); } -ErrorOr> Directory::open(StringView filename, Stream::OpenMode mode) const +ErrorOr> Directory::open(StringView filename, File::OpenMode mode) const { - auto fd = TRY(System::openat(m_directory_fd, filename, Stream::File::open_mode_to_options(mode))); - return Stream::File::adopt_fd(fd, mode); + auto fd = TRY(System::openat(m_directory_fd, filename, File::open_mode_to_options(mode))); + return File::adopt_fd(fd, mode); } ErrorOr Directory::stat() const diff --git a/Userland/Libraries/LibCore/Directory.h b/Userland/Libraries/LibCore/Directory.h index 8518c991e31f50..c5ef10caf6985e 100644 --- a/Userland/Libraries/LibCore/Directory.h +++ b/Userland/Libraries/LibCore/Directory.h @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -37,7 +38,7 @@ class Directory { static ErrorOr create(DeprecatedString path, CreateDirectories, mode_t creation_mode = 0755); static ErrorOr adopt_fd(int fd, Optional path = {}); - ErrorOr> open(StringView filename, Stream::OpenMode mode) const; + ErrorOr> open(StringView filename, File::OpenMode mode) const; ErrorOr stat() const; ErrorOr create_iterator() const; diff --git a/Userland/Libraries/LibCore/Stream.cpp b/Userland/Libraries/LibCore/File.cpp similarity index 93% rename from Userland/Libraries/LibCore/Stream.cpp rename to Userland/Libraries/LibCore/File.cpp index bb929ed9112d9b..d9eba43da0cf11 100644 --- a/Userland/Libraries/LibCore/Stream.cpp +++ b/Userland/Libraries/LibCore/File.cpp @@ -5,22 +5,12 @@ * SPDX-License-Identifier: BSD-2-Clause */ -#include "Stream.h" +#include #include #include -#include -#include -#include -#include #include -#ifdef AK_OS_SERENITY -# include -#endif -#ifdef AK_OS_FREEBSD -# include -#endif -namespace Core::Stream { +namespace Core { ErrorOr> File::open(StringView filename, OpenMode mode, mode_t permissions) { @@ -36,7 +26,7 @@ ErrorOr> File::adopt_fd(int fd, OpenMode mode, ShouldCloseFi } if (!has_any_flag(mode, OpenMode::ReadWrite)) { - dbgln("Core::DeprecatedFile::adopt_fd: Attempting to adopt a file with neither Read nor Write specified in mode"); + dbgln("Core::File::adopt_fd: Attempting to adopt a file with neither Read nor Write specified in mode"); return Error::from_errno(EINVAL); } diff --git a/Userland/Libraries/LibCore/File.h b/Userland/Libraries/LibCore/File.h new file mode 100644 index 00000000000000..dce8fd78e97bb5 --- /dev/null +++ b/Userland/Libraries/LibCore/File.h @@ -0,0 +1,108 @@ +/* + * Copyright (c) 2021, sin-ack + * Copyright (c) 2022, the SerenityOS developers. + * + * SPDX-License-Identifier: BSD-2-Clause + */ + +#pragma once + +#include +#include +#include +#include +#include +#include + +namespace Core { + +class File final : public SeekableStream { + AK_MAKE_NONCOPYABLE(File); + +public: + enum class OpenMode : unsigned { + NotOpen = 0, + Read = 1, + Write = 2, + ReadWrite = 3, + Append = 4, + Truncate = 8, + MustBeNew = 16, + KeepOnExec = 32, + Nonblocking = 64, + }; + + enum class ShouldCloseFileDescriptor { + Yes, + No, + }; + + static ErrorOr> open(StringView filename, OpenMode, mode_t = 0644); + static ErrorOr> adopt_fd(int fd, OpenMode, ShouldCloseFileDescriptor = ShouldCloseFileDescriptor::Yes); + + static ErrorOr> standard_input(); + static ErrorOr> standard_output(); + static ErrorOr> standard_error(); + static ErrorOr> open_file_or_standard_stream(StringView filename, OpenMode mode); + + File(File&& other) { operator=(move(other)); } + + File& operator=(File&& other) + { + if (&other == this) + return *this; + + m_mode = exchange(other.m_mode, OpenMode::NotOpen); + m_fd = exchange(other.m_fd, -1); + m_last_read_was_eof = exchange(other.m_last_read_was_eof, false); + return *this; + } + + virtual ErrorOr read(Bytes) override; + virtual ErrorOr read_until_eof(size_t block_size = 4096) override; + virtual ErrorOr write(ReadonlyBytes) override; + virtual bool is_eof() const override; + virtual bool is_open() const override; + virtual void close() override; + virtual ErrorOr seek(i64 offset, SeekMode) override; + virtual ErrorOr truncate(size_t length) override; + + int leak_fd(Badge<::IPC::File>) + { + m_should_close_file_descriptor = ShouldCloseFileDescriptor::No; + return m_fd; + } + + int fd() const + { + return m_fd; + } + + virtual ~File() override + { + if (m_should_close_file_descriptor == ShouldCloseFileDescriptor::Yes) + close(); + } + + static int open_mode_to_options(OpenMode mode); + +private: + File(OpenMode mode, ShouldCloseFileDescriptor should_close = ShouldCloseFileDescriptor::Yes) + : m_mode(mode) + , m_should_close_file_descriptor(should_close) + { + } + + ErrorOr open_path(StringView filename, mode_t); + + OpenMode m_mode { OpenMode::NotOpen }; + int m_fd { -1 }; + bool m_last_read_was_eof { false }; + ShouldCloseFileDescriptor m_should_close_file_descriptor { ShouldCloseFileDescriptor::Yes }; +}; + +AK_ENUM_BITWISE_OPERATORS(File::OpenMode) + +using BufferedFile = BufferedSeekable; + +} diff --git a/Userland/Libraries/LibCore/Forward.h b/Userland/Libraries/LibCore/Forward.h index dff8a2aad3dfcf..a0e81ec8859591 100644 --- a/Userland/Libraries/LibCore/Forward.h +++ b/Userland/Libraries/LibCore/Forward.h @@ -21,6 +21,7 @@ class DeferredInvocationContext; class ElapsedTimer; class Event; class EventLoop; +class File; class IODevice; class LocalServer; class LocalSocket; @@ -42,8 +43,4 @@ class UDPSocket; enum class TimerShouldFireWhenNotVisible; -namespace Stream { -class File; -} - } diff --git a/Userland/Libraries/LibCore/ProcessStatisticsReader.cpp b/Userland/Libraries/LibCore/ProcessStatisticsReader.cpp index cb6726cc4466cf..ccb423e7c7ea2e 100644 --- a/Userland/Libraries/LibCore/ProcessStatisticsReader.cpp +++ b/Userland/Libraries/LibCore/ProcessStatisticsReader.cpp @@ -8,6 +8,7 @@ #include #include #include +#include #include #include @@ -90,7 +91,7 @@ ErrorOr ProcessStatisticsReader::get_all(SeekableStream& ErrorOr ProcessStatisticsReader::get_all(bool include_usernames) { - auto proc_all_file = TRY(Core::Stream::File::open("/sys/kernel/processes"sv, Core::Stream::OpenMode::Read)); + auto proc_all_file = TRY(Core::File::open("/sys/kernel/processes"sv, Core::File::OpenMode::Read)); return get_all(*proc_all_file, include_usernames); } diff --git a/Userland/Libraries/LibCore/Stream.h b/Userland/Libraries/LibCore/Stream.h index 791257c127e7e1..93f5cd6d67e293 100644 --- a/Userland/Libraries/LibCore/Stream.h +++ b/Userland/Libraries/LibCore/Stream.h @@ -25,98 +25,3 @@ #include #include #include - -namespace Core::Stream { - -// Concrete classes. - -enum class OpenMode : unsigned { - NotOpen = 0, - Read = 1, - Write = 2, - ReadWrite = 3, - Append = 4, - Truncate = 8, - MustBeNew = 16, - KeepOnExec = 32, - Nonblocking = 64, -}; - -enum class ShouldCloseFileDescriptor { - Yes, - No, -}; - -AK_ENUM_BITWISE_OPERATORS(OpenMode) - -class File final : public SeekableStream { - AK_MAKE_NONCOPYABLE(File); - -public: - static ErrorOr> open(StringView filename, OpenMode, mode_t = 0644); - static ErrorOr> adopt_fd(int fd, OpenMode, ShouldCloseFileDescriptor = ShouldCloseFileDescriptor::Yes); - - static ErrorOr> standard_input(); - static ErrorOr> standard_output(); - static ErrorOr> standard_error(); - static ErrorOr> open_file_or_standard_stream(StringView filename, OpenMode mode); - - File(File&& other) { operator=(move(other)); } - - File& operator=(File&& other) - { - if (&other == this) - return *this; - - m_mode = exchange(other.m_mode, OpenMode::NotOpen); - m_fd = exchange(other.m_fd, -1); - m_last_read_was_eof = exchange(other.m_last_read_was_eof, false); - return *this; - } - - virtual ErrorOr read(Bytes) override; - virtual ErrorOr read_until_eof(size_t block_size = 4096) override; - virtual ErrorOr write(ReadonlyBytes) override; - virtual bool is_eof() const override; - virtual bool is_open() const override; - virtual void close() override; - virtual ErrorOr seek(i64 offset, SeekMode) override; - virtual ErrorOr truncate(size_t length) override; - - int leak_fd(Badge<::IPC::File>) - { - m_should_close_file_descriptor = ShouldCloseFileDescriptor::No; - return m_fd; - } - - int fd() const - { - return m_fd; - } - - virtual ~File() override - { - if (m_should_close_file_descriptor == ShouldCloseFileDescriptor::Yes) - close(); - } - - static int open_mode_to_options(OpenMode mode); - -private: - File(OpenMode mode, ShouldCloseFileDescriptor should_close = ShouldCloseFileDescriptor::Yes) - : m_mode(mode) - , m_should_close_file_descriptor(should_close) - { - } - - ErrorOr open_path(StringView filename, mode_t); - - OpenMode m_mode { OpenMode::NotOpen }; - int m_fd { -1 }; - bool m_last_read_was_eof { false }; - ShouldCloseFileDescriptor m_should_close_file_descriptor { ShouldCloseFileDescriptor::Yes }; -}; - -using BufferedFile = BufferedSeekable; - -} diff --git a/Userland/Libraries/LibFileSystemAccessClient/Client.cpp b/Userland/Libraries/LibFileSystemAccessClient/Client.cpp index ea57978b0b6d6c..05cdd029d2980c 100644 --- a/Userland/Libraries/LibFileSystemAccessClient/Client.cpp +++ b/Userland/Libraries/LibFileSystemAccessClient/Client.cpp @@ -72,23 +72,23 @@ Result Client::request_file_read_only_approved(GUI::Window* parent_window, Depre return handle_promise(id); } -static Core::Stream::OpenMode to_stream_open_mode(Core::OpenMode open_mode) +static Core::File::OpenMode to_stream_open_mode(Core::OpenMode open_mode) { - Core::Stream::OpenMode result {}; + Core::File::OpenMode result {}; if ((open_mode & Core::OpenMode::ReadOnly) == Core::OpenMode::ReadOnly) - result |= Core::Stream::OpenMode::Read; + result |= Core::File::OpenMode::Read; if ((open_mode & Core::OpenMode::WriteOnly) == Core::OpenMode::WriteOnly) - result |= Core::Stream::OpenMode::Write; + result |= Core::File::OpenMode::Write; if ((open_mode & Core::OpenMode::ReadWrite) == Core::OpenMode::ReadWrite) - result |= Core::Stream::OpenMode::ReadWrite; + result |= Core::File::OpenMode::ReadWrite; if ((open_mode & Core::OpenMode::Append) == Core::OpenMode::Append) - result |= Core::Stream::OpenMode::Append; + result |= Core::File::OpenMode::Append; if ((open_mode & Core::OpenMode::Truncate) == Core::OpenMode::Truncate) - result |= Core::Stream::OpenMode::Truncate; + result |= Core::File::OpenMode::Truncate; if ((open_mode & Core::OpenMode::MustBeNew) == Core::OpenMode::MustBeNew) - result |= Core::Stream::OpenMode::MustBeNew; + result |= Core::File::OpenMode::MustBeNew; if ((open_mode & Core::OpenMode::KeepOnExec) == Core::OpenMode::KeepOnExec) - result |= Core::Stream::OpenMode::KeepOnExec; + result |= Core::File::OpenMode::KeepOnExec; return result; } @@ -120,7 +120,7 @@ DeprecatedResult Client::try_request_file_deprecated(GUI::Window* parent_window, return handle_promise(id); } -Result Client::request_file(GUI::Window* parent_window, DeprecatedString const& path, Core::Stream::OpenMode mode) +Result Client::request_file(GUI::Window* parent_window, DeprecatedString const& path, Core::File::OpenMode mode) { auto const id = get_new_id(); m_promises.set(id, PromiseAndWindow { { Core::Promise::construct() }, parent_window }); @@ -167,7 +167,7 @@ DeprecatedResult Client::try_open_file_deprecated(GUI::Window* parent_window, De return handle_promise(id); } -Result Client::open_file(GUI::Window* parent_window, DeprecatedString const& window_title, StringView path, Core::Stream::OpenMode requested_access) +Result Client::open_file(GUI::Window* parent_window, DeprecatedString const& window_title, StringView path, Core::File::OpenMode requested_access) { auto const id = get_new_id(); m_promises.set(id, PromiseAndWindow { { Core::Promise::construct() }, parent_window }); @@ -209,7 +209,7 @@ DeprecatedResult Client::try_save_file_deprecated(GUI::Window* parent_window, De return handle_promise(id); } -Result Client::save_file(GUI::Window* parent_window, DeprecatedString const& name, DeprecatedString const ext, Core::Stream::OpenMode requested_access) +Result Client::save_file(GUI::Window* parent_window, DeprecatedString const& name, DeprecatedString const ext, Core::File::OpenMode requested_access) { auto const id = get_new_id(); m_promises.set(id, PromiseAndWindow { { Core::Promise::construct() }, parent_window }); @@ -275,7 +275,7 @@ void Client::handle_prompt_end(i32 request_id, i32 error, Optional co } auto file_or_error = [&]() -> ErrorOr { - auto stream = TRY(Core::Stream::File::adopt_fd(ipc_file->take_fd(), Core::Stream::OpenMode::ReadWrite)); + auto stream = TRY(Core::File::adopt_fd(ipc_file->take_fd(), Core::File::OpenMode::ReadWrite)); auto filename = TRY(String::from_deprecated_string(*chosen_file)); return File({}, move(stream), filename); }(); diff --git a/Userland/Libraries/LibFileSystemAccessClient/Client.h b/Userland/Libraries/LibFileSystemAccessClient/Client.h index d44d136d599fe6..c5e5f18418dfa0 100644 --- a/Userland/Libraries/LibFileSystemAccessClient/Client.h +++ b/Userland/Libraries/LibFileSystemAccessClient/Client.h @@ -22,18 +22,18 @@ namespace FileSystemAccessClient { class Client; class File { public: - File(Badge, NonnullOwnPtr stream, String filename) + File(Badge, NonnullOwnPtr stream, String filename) : m_stream(move(stream)) , m_filename(filename) { } - Core::Stream::File& stream() const { return *m_stream; } - NonnullOwnPtr release_stream() { return move(m_stream); } + Core::File& stream() const { return *m_stream; } + NonnullOwnPtr release_stream() { return move(m_stream); } String filename() const { return m_filename; } private: - NonnullOwnPtr m_stream; + NonnullOwnPtr m_stream; String m_filename; }; @@ -52,9 +52,9 @@ class Client final DeprecatedResult try_save_file_deprecated(GUI::Window* parent_window, DeprecatedString const& name, DeprecatedString const ext, Core::OpenMode requested_access = Core::OpenMode::WriteOnly | Core::OpenMode::Truncate); Result request_file_read_only_approved(GUI::Window* parent_window, DeprecatedString const& path); - Result request_file(GUI::Window* parent_window, DeprecatedString const& path, Core::Stream::OpenMode requested_access); - Result open_file(GUI::Window* parent_window, DeprecatedString const& window_title = {}, StringView path = Core::StandardPaths::home_directory(), Core::Stream::OpenMode requested_access = Core::Stream::OpenMode::Read); - Result save_file(GUI::Window* parent_window, DeprecatedString const& name, DeprecatedString const ext, Core::Stream::OpenMode requested_access = Core::Stream::OpenMode::Write | Core::Stream::OpenMode::Truncate); + Result request_file(GUI::Window* parent_window, DeprecatedString const& path, Core::File::OpenMode requested_access); + Result open_file(GUI::Window* parent_window, DeprecatedString const& window_title = {}, StringView path = Core::StandardPaths::home_directory(), Core::File::OpenMode requested_access = Core::File::OpenMode::Read); + Result save_file(GUI::Window* parent_window, DeprecatedString const& name, DeprecatedString const ext, Core::File::OpenMode requested_access = Core::File::OpenMode::Write | Core::File::OpenMode::Truncate); static Client& the(); diff --git a/Userland/Libraries/LibGUI/AbstractThemePreview.cpp b/Userland/Libraries/LibGUI/AbstractThemePreview.cpp index df70abeb012f07..e3c65230483428 100644 --- a/Userland/Libraries/LibGUI/AbstractThemePreview.cpp +++ b/Userland/Libraries/LibGUI/AbstractThemePreview.cpp @@ -84,7 +84,7 @@ void AbstractThemePreview::set_theme(Core::AnonymousBuffer const& theme_buffer) set_preview_palette(m_preview_palette); } -ErrorOr AbstractThemePreview::set_theme_from_file(StringView path, NonnullOwnPtr file) +ErrorOr AbstractThemePreview::set_theme_from_file(StringView path, NonnullOwnPtr file) { auto config_file = TRY(Core::ConfigFile::open(path, move(file))); auto theme = TRY(Gfx::load_system_theme(config_file)); diff --git a/Userland/Libraries/LibGUI/AbstractThemePreview.h b/Userland/Libraries/LibGUI/AbstractThemePreview.h index 5800262133b9b2..203c4084064112 100644 --- a/Userland/Libraries/LibGUI/AbstractThemePreview.h +++ b/Userland/Libraries/LibGUI/AbstractThemePreview.h @@ -24,7 +24,7 @@ class AbstractThemePreview : public GUI::Frame { Gfx::Palette const& preview_palette() const { return m_preview_palette; } void set_preview_palette(Gfx::Palette const&); - ErrorOr set_theme_from_file(StringView path, NonnullOwnPtr); + ErrorOr set_theme_from_file(StringView path, NonnullOwnPtr); void set_theme(Core::AnonymousBuffer const&); void paint_window(StringView title, Gfx::IntRect const& rect, Gfx::WindowTheme::WindowState, Gfx::Bitmap const& icon, int button_count = 3); diff --git a/Userland/Libraries/LibGUI/CommonLocationsProvider.cpp b/Userland/Libraries/LibGUI/CommonLocationsProvider.cpp index 9beebbf0056f28..99186d1d15e580 100644 --- a/Userland/Libraries/LibGUI/CommonLocationsProvider.cpp +++ b/Userland/Libraries/LibGUI/CommonLocationsProvider.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -42,7 +43,7 @@ static void initialize_if_needed() ErrorOr CommonLocationsProvider::load_from_json(StringView json_path) { - auto file = TRY(Core::Stream::File::open(json_path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(json_path, Core::File::OpenMode::Read)); auto json = JsonValue::from_string(TRY(file->read_until_eof())); if (json.is_error()) return Error::from_string_literal("File is not a valid JSON"); diff --git a/Userland/Libraries/LibGUI/TextEditor.cpp b/Userland/Libraries/LibGUI/TextEditor.cpp index 622a8902eeaf63..4edf15466281a1 100644 --- a/Userland/Libraries/LibGUI/TextEditor.cpp +++ b/Userland/Libraries/LibGUI/TextEditor.cpp @@ -1460,12 +1460,12 @@ void TextEditor::timer_event(Core::TimerEvent&) ErrorOr TextEditor::write_to_file(StringView path) { - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Write | Core::Stream::OpenMode::Truncate)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Write | Core::File::OpenMode::Truncate)); TRY(write_to_file(*file)); return {}; } -ErrorOr TextEditor::write_to_file(Core::Stream::File& file) +ErrorOr TextEditor::write_to_file(Core::File& file) { off_t file_size = 0; if (line_count() == 1 && line(0).is_empty()) { diff --git a/Userland/Libraries/LibGUI/TextEditor.h b/Userland/Libraries/LibGUI/TextEditor.h index 34435a0a0ec771..b5bce2780748aa 100644 --- a/Userland/Libraries/LibGUI/TextEditor.h +++ b/Userland/Libraries/LibGUI/TextEditor.h @@ -130,7 +130,7 @@ class TextEditor void insert_at_cursor_or_replace_selection(StringView); void replace_all_text_without_resetting_undo_stack(StringView text); ErrorOr write_to_file(StringView path); - ErrorOr write_to_file(Core::Stream::File&); + ErrorOr write_to_file(Core::File&); bool has_selection() const { return m_selection.is_valid(); } DeprecatedString selected_text() const; size_t number_of_words() const; diff --git a/Userland/Libraries/LibGfx/Font/BitmapFont.cpp b/Userland/Libraries/LibGfx/Font/BitmapFont.cpp index 6aabeefebb35c1..5db821d27bcb8c 100644 --- a/Userland/Libraries/LibGfx/Font/BitmapFont.cpp +++ b/Userland/Libraries/LibGfx/Font/BitmapFont.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -257,7 +258,7 @@ ErrorOr BitmapFont::write_to_file(DeprecatedString const& path) memcpy(header.name, m_name.characters(), min(m_name.length(), sizeof(header.name) - 1)); memcpy(header.family, m_family.characters(), min(m_family.length(), sizeof(header.family) - 1)); - auto stream = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Write)); + auto stream = TRY(Core::File::open(path, Core::File::OpenMode::Write)); size_t bytes_per_glyph = sizeof(u32) * m_glyph_height; TRY(stream->write_entire_buffer({ &header, sizeof(header) })); TRY(stream->write_entire_buffer({ m_range_mask, m_range_mask_size })); diff --git a/Userland/Libraries/LibIDL/IDLParser.cpp b/Userland/Libraries/LibIDL/IDLParser.cpp index 2fe98cdd2118f3..1e2dabb01e3fbc 100644 --- a/Userland/Libraries/LibIDL/IDLParser.cpp +++ b/Userland/Libraries/LibIDL/IDLParser.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include [[noreturn]] static void report_parsing_error(StringView message, StringView filename, StringView input, size_t offset) @@ -149,7 +150,7 @@ Optional Parser::resolve_import(auto path) report_parsing_error(DeprecatedString::formatted("Circular import detected: {}", include_path), filename, input, lexer.tell()); import_stack.set(real_path); - auto file_or_error = Core::Stream::File::open(real_path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(real_path, Core::File::OpenMode::Read); if (file_or_error.is_error()) report_parsing_error(DeprecatedString::formatted("Failed to open {}: {}", real_path, file_or_error.error()), filename, input, lexer.tell()); diff --git a/Userland/Libraries/LibIPC/File.h b/Userland/Libraries/LibIPC/File.h index 09453b81491b1a..d19156709ef931 100644 --- a/Userland/Libraries/LibIPC/File.h +++ b/Userland/Libraries/LibIPC/File.h @@ -9,6 +9,7 @@ #include #include +#include #include #include @@ -41,7 +42,7 @@ class File { } template - File(Core::Stream::File& file, Args... args) + File(Core::File& file, Args... args) : File(file.leak_fd(Badge {}), args...) { } diff --git a/Userland/Libraries/LibLine/Editor.cpp b/Userland/Libraries/LibLine/Editor.cpp index ad165c79c3614e..722df187b5d8ce 100644 --- a/Userland/Libraries/LibLine/Editor.cpp +++ b/Userland/Libraries/LibLine/Editor.cpp @@ -602,7 +602,7 @@ ErrorOr Editor::interrupted() m_finish = false; { - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(reposition_cursor(*stderr_stream, true)); if (TRY(m_suggestion_display->cleanup())) TRY(reposition_cursor(*stderr_stream, true)); @@ -648,7 +648,7 @@ ErrorOr Editor::handle_resize_event(bool reset_origin) set_origin(m_origin_row, 1); - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(reposition_cursor(*stderr_stream, true)); TRY(m_suggestion_display->redisplay(m_suggestion_manager, m_num_lines, m_num_columns)); @@ -665,7 +665,7 @@ ErrorOr Editor::really_quit_event_loop() { m_finish = false; { - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(reposition_cursor(*stderr_stream, true)); TRY(stderr_stream->write_entire_buffer("\n"sv.bytes())); } @@ -731,7 +731,7 @@ auto Editor::get_line(DeprecatedString const& prompt) -> Resultwrite_entire_buffer("\n"sv.bytes()).release_value_but_fixme_should_propagate_errors(); @@ -1173,7 +1173,7 @@ ErrorOr Editor::handle_read_event() for (auto& view : completion_result.insert) insert(view); - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(reposition_cursor(*stderr_stream)); if (completion_result.style_to_apply.has_value()) { @@ -1252,7 +1252,7 @@ ErrorOr Editor::cleanup_suggestions() // We probably have some suggestions drawn, // let's clean them up. if (TRY(m_suggestion_display->cleanup())) { - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(reposition_cursor(*stderr_stream)); m_refresh_needed = true; } @@ -1326,7 +1326,7 @@ ErrorOr Editor::cleanup() if (new_lines < m_shown_lines) m_extra_forward_lines = max(m_shown_lines - new_lines, m_extra_forward_lines); - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(reposition_cursor(*stderr_stream, true)); auto current_line = num_lines() - 1; TRY(VT::clear_lines(current_line, m_extra_forward_lines, *stderr_stream)); diff --git a/Userland/Libraries/LibLine/InternalFunctions.cpp b/Userland/Libraries/LibLine/InternalFunctions.cpp index 32e520baf5c47e..5b137debd75b9c 100644 --- a/Userland/Libraries/LibLine/InternalFunctions.cpp +++ b/Userland/Libraries/LibLine/InternalFunctions.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -341,7 +342,7 @@ void Editor::enter_search() auto& search_string = search_string_result.value(); // Manually cleanup the search line. - auto stderr_stream = Core::Stream::File::standard_error().release_value_but_fixme_should_propagate_errors(); + auto stderr_stream = Core::File::standard_error().release_value_but_fixme_should_propagate_errors(); reposition_cursor(*stderr_stream).release_value_but_fixme_should_propagate_errors(); auto search_metrics = actual_rendered_string_metrics(search_string, {}); auto metrics = actual_rendered_string_metrics(search_prompt, {}); @@ -433,7 +434,7 @@ void Editor::go_end() void Editor::clear_screen() { warn("\033[3J\033[H\033[2J"); - auto stream = Core::Stream::File::standard_error().release_value_but_fixme_should_propagate_errors(); + auto stream = Core::File::standard_error().release_value_but_fixme_should_propagate_errors(); VT::move_absolute(1, 1, *stream).release_value_but_fixme_should_propagate_errors(); set_origin(1, 1); m_refresh_needed = true; @@ -528,7 +529,7 @@ void Editor::edit_in_external_editor() { auto write_fd = dup(fd); - auto stream = Core::Stream::File::adopt_fd(write_fd, Core::Stream::OpenMode::Write).release_value_but_fixme_should_propagate_errors(); + auto stream = Core::File::adopt_fd(write_fd, Core::File::OpenMode::Write).release_value_but_fixme_should_propagate_errors(); StringBuilder builder; builder.append(Utf32View { m_buffer.data(), m_buffer.size() }); auto bytes = builder.string_view().bytes(); @@ -569,7 +570,7 @@ void Editor::edit_in_external_editor() } { - auto file = Core::Stream::File::open({ file_path, strlen(file_path) }, Core::Stream::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); + auto file = Core::File::open({ file_path, strlen(file_path) }, Core::File::OpenMode::Read).release_value_but_fixme_should_propagate_errors(); auto contents = file->read_until_eof().release_value_but_fixme_should_propagate_errors(); StringView data { contents }; while (data.ends_with('\n')) diff --git a/Userland/Libraries/LibLine/XtermSuggestionDisplay.cpp b/Userland/Libraries/LibLine/XtermSuggestionDisplay.cpp index ace6d42f078887..5a51e951834545 100644 --- a/Userland/Libraries/LibLine/XtermSuggestionDisplay.cpp +++ b/Userland/Libraries/LibLine/XtermSuggestionDisplay.cpp @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -17,7 +18,7 @@ ErrorOr XtermSuggestionDisplay::display(SuggestionManager const& manager) { did_display(); - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); size_t longest_suggestion_length = 0; size_t longest_suggestion_byte_length = 0; @@ -161,7 +162,7 @@ ErrorOr XtermSuggestionDisplay::cleanup() did_cleanup(); if (m_lines_used_for_last_suggestions) { - auto stderr_stream = TRY(Core::Stream::File::standard_error()); + auto stderr_stream = TRY(Core::File::standard_error()); TRY(VT::clear_lines(0, m_lines_used_for_last_suggestions, *stderr_stream)); m_lines_used_for_last_suggestions = 0; return true; diff --git a/Userland/Libraries/LibProtocol/Request.cpp b/Userland/Libraries/LibProtocol/Request.cpp index 882372217da5c9..d533c33ec88c3b 100644 --- a/Userland/Libraries/LibProtocol/Request.cpp +++ b/Userland/Libraries/LibProtocol/Request.cpp @@ -24,7 +24,7 @@ void Request::stream_into(AK::Stream& stream) { VERIFY(!m_internal_stream_data); - m_internal_stream_data = make(MUST(Core::Stream::File::adopt_fd(fd(), Core::Stream::OpenMode::Read))); + m_internal_stream_data = make(MUST(Core::File::adopt_fd(fd(), Core::File::OpenMode::Read))); m_internal_stream_data->read_notifier = Core::Notifier::construct(fd(), Core::Notifier::Read); auto user_on_finish = move(on_finish); diff --git a/Userland/Libraries/LibSQL/Heap.cpp b/Userland/Libraries/LibSQL/Heap.cpp index 6ad28a51d0f49d..3dff07f8007df7 100644 --- a/Userland/Libraries/LibSQL/Heap.cpp +++ b/Userland/Libraries/LibSQL/Heap.cpp @@ -47,8 +47,8 @@ ErrorOr Heap::open() if (file_size > 0) m_next_block = m_end_of_file = file_size / BLOCKSIZE; - auto file = TRY(Core::Stream::File::open(name(), Core::Stream::OpenMode::ReadWrite)); - m_file = TRY(Core::Stream::BufferedFile::create(move(file))); + auto file = TRY(Core::File::open(name(), Core::File::OpenMode::ReadWrite)); + m_file = TRY(Core::BufferedFile::create(move(file))); if (file_size > 0) { if (auto error_maybe = read_zero_block(); error_maybe.is_error()) { diff --git a/Userland/Libraries/LibSQL/Heap.h b/Userland/Libraries/LibSQL/Heap.h index 492141fa08a8fc..fa1a2cc9fe4290 100644 --- a/Userland/Libraries/LibSQL/Heap.h +++ b/Userland/Libraries/LibSQL/Heap.h @@ -11,6 +11,7 @@ #include #include #include +#include #include #include @@ -99,7 +100,7 @@ class Heap : public Core::Object { void initialize_zero_block(); void update_zero_block(); - OwnPtr m_file; + OwnPtr m_file; u32 m_free_list { 0 }; u32 m_next_block { 1 }; u32 m_end_of_file { 1 }; diff --git a/Userland/Libraries/LibSQL/SQLClient.cpp b/Userland/Libraries/LibSQL/SQLClient.cpp index 36236487ed2c2f..08c9ec8c2d058d 100644 --- a/Userland/Libraries/LibSQL/SQLClient.cpp +++ b/Userland/Libraries/LibSQL/SQLClient.cpp @@ -67,7 +67,7 @@ static ErrorOr launch_server(DeprecatedString const& socket_path, Deprecat server_pid = TRY(Core::System::fork()); if (server_pid != 0) { - auto server_pid_file = TRY(Core::Stream::File::open(pid_path, Core::Stream::OpenMode::Write)); + auto server_pid_file = TRY(Core::File::open(pid_path, Core::File::OpenMode::Write)); TRY(server_pid_file->write(DeprecatedString::number(server_pid).bytes())); TRY(Core::System::kill(getpid(), SIGTERM)); @@ -108,7 +108,7 @@ static ErrorOr should_launch_server(DeprecatedString const& pid_path) Optional pid; { - auto server_pid_file = Core::Stream::File::open(pid_path, Core::Stream::OpenMode::Read); + auto server_pid_file = Core::File::open(pid_path, Core::File::OpenMode::Read); if (server_pid_file.is_error()) { warnln("Could not open SQLServer PID file '{}': {}", pid_path, server_pid_file.error()); return server_pid_file.release_error(); diff --git a/Userland/Libraries/LibTLS/HandshakeClient.cpp b/Userland/Libraries/LibTLS/HandshakeClient.cpp index 55287b7df7d01b..7055531892e5fe 100644 --- a/Userland/Libraries/LibTLS/HandshakeClient.cpp +++ b/Userland/Libraries/LibTLS/HandshakeClient.cpp @@ -140,7 +140,7 @@ bool TLSv12::compute_master_secret_from_pre_master_secret(size_t length) } if constexpr (TLS_SSL_KEYLOG_DEBUG) { - auto file = MUST(Core::Stream::File::open("/home/anon/ssl_keylog"sv, Core::Stream::OpenMode::Append | Core::Stream::OpenMode::Write)); + auto file = MUST(Core::File::open("/home/anon/ssl_keylog"sv, Core::File::OpenMode::Append | Core::File::OpenMode::Write)); MUST(file->write_entire_buffer("CLIENT_RANDOM "sv.bytes())); MUST(file->write_entire_buffer(encode_hex({ m_context.local_random, 32 }).bytes())); MUST(file->write_entire_buffer(" "sv.bytes())); diff --git a/Userland/Libraries/LibTest/JavaScriptTestRunner.h b/Userland/Libraries/LibTest/JavaScriptTestRunner.h index 997802532a8270..7263612604653d 100644 --- a/Userland/Libraries/LibTest/JavaScriptTestRunner.h +++ b/Userland/Libraries/LibTest/JavaScriptTestRunner.h @@ -17,6 +17,7 @@ #include #include #include +#include #include #include #include @@ -217,7 +218,7 @@ inline JS::ThrowCompletionOr TestRunnerGlobalObject::initialize(JS::Realm& inline ByteBuffer load_entire_file(StringView path) { auto try_load_entire_file = [](StringView const& path) -> ErrorOr { - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto file_size = TRY(file->size()); auto content = TRY(ByteBuffer::create_uninitialized(file_size)); TRY(file->read(content.bytes())); diff --git a/Userland/Libraries/LibVirtGPU/Device.cpp b/Userland/Libraries/LibVirtGPU/Device.cpp index f1ad7135804e7b..34c2bf9de80fdc 100644 --- a/Userland/Libraries/LibVirtGPU/Device.cpp +++ b/Userland/Libraries/LibVirtGPU/Device.cpp @@ -8,6 +8,7 @@ #include #include +#include #include #include #include @@ -38,14 +39,14 @@ static constexpr auto vert_shader = "VERT\n" " 4: MOV_SAT OUT[1], IN[1]\n" " 5: END\n"sv; -Device::Device(NonnullOwnPtr gpu_file) +Device::Device(NonnullOwnPtr gpu_file) : m_gpu_file { move(gpu_file) } { } ErrorOr> Device::create(Gfx::IntSize min_size) { - auto file = TRY(Core::Stream::File::open("/dev/gpu/render0"sv, Core::Stream::OpenMode::ReadWrite)); + auto file = TRY(Core::File::open("/dev/gpu/render0"sv, Core::File::OpenMode::ReadWrite)); auto device = make(move(file)); TRY(device->initialize_context(min_size)); return device; diff --git a/Userland/Libraries/LibVirtGPU/Device.h b/Userland/Libraries/LibVirtGPU/Device.h index 8f5d1308713cb1..ee7c31db18c43c 100644 --- a/Userland/Libraries/LibVirtGPU/Device.h +++ b/Userland/Libraries/LibVirtGPU/Device.h @@ -18,7 +18,7 @@ namespace VirtGPU { class Device final : public GPU::Device { public: - Device(NonnullOwnPtr); + Device(NonnullOwnPtr); static ErrorOr> create(Gfx::IntSize min_size); @@ -66,7 +66,7 @@ class Device final : public GPU::Device { ErrorOr create_virgl_resource(VirGL3DResourceSpec&); ErrorOr upload_command_buffer(Vector const&); - NonnullOwnPtr m_gpu_file; + NonnullOwnPtr m_gpu_file; Protocol::ResourceID m_vbo_resource_id { 0 }; Protocol::ResourceID m_drawtarget { 0 }; diff --git a/Userland/Libraries/LibWeb/Loader/ResourceLoader.cpp b/Userland/Libraries/LibWeb/Loader/ResourceLoader.cpp index ac104ff744537a..0e2b4876dabd25 100644 --- a/Userland/Libraries/LibWeb/Loader/ResourceLoader.cpp +++ b/Userland/Libraries/LibWeb/Loader/ResourceLoader.cpp @@ -250,7 +250,7 @@ void ResourceLoader::load(LoadRequest& request, Function #include #include +#include #include #include #include @@ -38,7 +39,7 @@ static ErrorOr perform_copy(Vector const& sources, DeprecatedSt static ErrorOr perform_move(Vector const& sources, DeprecatedString const& destination); static ErrorOr perform_delete(Vector const& sources); static ErrorOr execute_work_items(Vector const& items); -static ErrorOr> open_destination_file(DeprecatedString const& destination); +static ErrorOr> open_destination_file(DeprecatedString const& destination); static DeprecatedString deduplicate_destination_file_name(DeprecatedString const& destination); ErrorOr serenity_main(Main::Arguments arguments) @@ -230,7 +231,7 @@ ErrorOr execute_work_items(Vector const& items) }; auto copy_file = [&](DeprecatedString const& source, DeprecatedString const& destination) -> ErrorOr { - auto source_file = TRY(Core::Stream::File::open(source, Core::Stream::OpenMode::Read)); + auto source_file = TRY(Core::File::open(source, Core::File::OpenMode::Read)); // FIXME: When the file already exists, let the user choose the next action instead of renaming it by default. auto destination_file = TRY(open_destination_file(destination)); auto buffer = TRY(ByteBuffer::create_zeroed(64 * KiB)); @@ -327,9 +328,9 @@ ErrorOr execute_work_items(Vector const& items) return 0; } -ErrorOr> open_destination_file(DeprecatedString const& destination) +ErrorOr> open_destination_file(DeprecatedString const& destination) { - auto destination_file_or_error = Core::Stream::File::open(destination, (Core::Stream::OpenMode)(Core::Stream::OpenMode::Write | Core::Stream::OpenMode::Truncate | Core::Stream::OpenMode::MustBeNew)); + auto destination_file_or_error = Core::File::open(destination, (Core::File::OpenMode)(Core::File::OpenMode::Write | Core::File::OpenMode::Truncate | Core::File::OpenMode::MustBeNew)); if (destination_file_or_error.is_error() && destination_file_or_error.error().code() == EEXIST) { return open_destination_file(deduplicate_destination_file_name(destination)); } diff --git a/Userland/Services/FileSystemAccessServer/ConnectionFromClient.cpp b/Userland/Services/FileSystemAccessServer/ConnectionFromClient.cpp index 255a7caf9e57ed..33b37ece8eb240 100644 --- a/Userland/Services/FileSystemAccessServer/ConnectionFromClient.cpp +++ b/Userland/Services/FileSystemAccessServer/ConnectionFromClient.cpp @@ -42,15 +42,15 @@ RefPtr ConnectionFromClient::create_dummy_child_window(i32 window_s return window; } -void ConnectionFromClient::request_file_handler(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& path, Core::Stream::OpenMode requested_access, ShouldPrompt prompt) +void ConnectionFromClient::request_file_handler(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& path, Core::File::OpenMode requested_access, ShouldPrompt prompt) { VERIFY(path.starts_with("/"sv)); bool approved = false; auto maybe_permissions = m_approved_files.get(path); - auto relevant_permissions = requested_access & (Core::Stream::OpenMode::Read | Core::Stream::OpenMode::Write); - VERIFY(relevant_permissions != Core::Stream::OpenMode::NotOpen); + auto relevant_permissions = requested_access & (Core::File::OpenMode::Read | Core::File::OpenMode::Write); + VERIFY(relevant_permissions != Core::File::OpenMode::NotOpen); if (maybe_permissions.has_value()) approved = has_flag(maybe_permissions.value(), relevant_permissions); @@ -58,11 +58,11 @@ void ConnectionFromClient::request_file_handler(i32 request_id, i32 window_serve if (!approved) { DeprecatedString access_string; - if (has_flag(requested_access, Core::Stream::OpenMode::ReadWrite)) + if (has_flag(requested_access, Core::File::OpenMode::ReadWrite)) access_string = "read and write"; - else if (has_flag(requested_access, Core::Stream::OpenMode::Read)) + else if (has_flag(requested_access, Core::File::OpenMode::Read)) access_string = "read from"; - else if (has_flag(requested_access, Core::Stream::OpenMode::Write)) + else if (has_flag(requested_access, Core::File::OpenMode::Write)) access_string = "write to"; auto pid = this->socket().peer_pid().release_value_but_fixme_should_propagate_errors(); @@ -90,7 +90,7 @@ void ConnectionFromClient::request_file_handler(i32 request_id, i32 window_serve } if (approved) { - auto file = Core::Stream::File::open(path, requested_access); + auto file = Core::File::open(path, requested_access); if (file.is_error()) { dbgln("FileSystemAccessServer: Couldn't open {}, error {}", path, file.error()); @@ -105,18 +105,18 @@ void ConnectionFromClient::request_file_handler(i32 request_id, i32 window_serve void ConnectionFromClient::request_file_read_only_approved(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& path) { - request_file_handler(request_id, window_server_client_id, parent_window_id, path, Core::Stream::OpenMode::Read, ShouldPrompt::No); + request_file_handler(request_id, window_server_client_id, parent_window_id, path, Core::File::OpenMode::Read, ShouldPrompt::No); } -void ConnectionFromClient::request_file(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& path, Core::Stream::OpenMode requested_access) +void ConnectionFromClient::request_file(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& path, Core::File::OpenMode requested_access) { request_file_handler(request_id, window_server_client_id, parent_window_id, path, requested_access, ShouldPrompt::Yes); } -void ConnectionFromClient::prompt_open_file(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& window_title, DeprecatedString const& path_to_view, Core::Stream::OpenMode requested_access) +void ConnectionFromClient::prompt_open_file(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& window_title, DeprecatedString const& path_to_view, Core::File::OpenMode requested_access) { - auto relevant_permissions = requested_access & (Core::Stream::OpenMode::Read | Core::Stream::OpenMode::Write); - VERIFY(relevant_permissions != Core::Stream::OpenMode::NotOpen); + auto relevant_permissions = requested_access & (Core::File::OpenMode::Read | Core::File::OpenMode::Write); + VERIFY(relevant_permissions != Core::File::OpenMode::NotOpen); auto main_window = create_dummy_child_window(window_server_client_id, parent_window_id); @@ -125,10 +125,10 @@ void ConnectionFromClient::prompt_open_file(i32 request_id, i32 window_server_cl prompt_helper(request_id, user_picked_file, requested_access); } -void ConnectionFromClient::prompt_save_file(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& name, DeprecatedString const& ext, DeprecatedString const& path_to_view, Core::Stream::OpenMode requested_access) +void ConnectionFromClient::prompt_save_file(i32 request_id, i32 window_server_client_id, i32 parent_window_id, DeprecatedString const& name, DeprecatedString const& ext, DeprecatedString const& path_to_view, Core::File::OpenMode requested_access) { - auto relevant_permissions = requested_access & (Core::Stream::OpenMode::Read | Core::Stream::OpenMode::Write); - VERIFY(relevant_permissions != Core::Stream::OpenMode::NotOpen); + auto relevant_permissions = requested_access & (Core::File::OpenMode::Read | Core::File::OpenMode::Write); + VERIFY(relevant_permissions != Core::File::OpenMode::NotOpen); auto main_window = create_dummy_child_window(window_server_client_id, parent_window_id); @@ -137,18 +137,18 @@ void ConnectionFromClient::prompt_save_file(i32 request_id, i32 window_server_cl prompt_helper(request_id, user_picked_file, requested_access); } -void ConnectionFromClient::prompt_helper(i32 request_id, Optional const& user_picked_file, Core::Stream::OpenMode requested_access) +void ConnectionFromClient::prompt_helper(i32 request_id, Optional const& user_picked_file, Core::File::OpenMode requested_access) { if (user_picked_file.has_value()) { VERIFY(user_picked_file->starts_with("/"sv)); - auto file = Core::Stream::File::open(user_picked_file.value(), requested_access); + auto file = Core::File::open(user_picked_file.value(), requested_access); if (file.is_error()) { dbgln("FileSystemAccessServer: Couldn't open {}, error {}", user_picked_file.value(), file.error()); async_handle_prompt_end(request_id, file.error().code(), Optional {}, user_picked_file); } else { auto maybe_permissions = m_approved_files.get(user_picked_file.value()); - auto new_permissions = requested_access & (Core::Stream::OpenMode::Read | Core::Stream::OpenMode::Write); + auto new_permissions = requested_access & (Core::File::OpenMode::Read | Core::File::OpenMode::Write); if (maybe_permissions.has_value()) new_permissions |= maybe_permissions.value(); diff --git a/Userland/Services/FileSystemAccessServer/ConnectionFromClient.h b/Userland/Services/FileSystemAccessServer/ConnectionFromClient.h index 584291ae30b52b..0ad767a3597ab9 100644 --- a/Userland/Services/FileSystemAccessServer/ConnectionFromClient.h +++ b/Userland/Services/FileSystemAccessServer/ConnectionFromClient.h @@ -28,22 +28,22 @@ class ConnectionFromClient final explicit ConnectionFromClient(NonnullOwnPtr); virtual void request_file_read_only_approved(i32, i32, i32, DeprecatedString const&) override; - virtual void request_file(i32, i32, i32, DeprecatedString const&, Core::Stream::OpenMode) override; - virtual void prompt_open_file(i32, i32, i32, DeprecatedString const&, DeprecatedString const&, Core::Stream::OpenMode) override; - virtual void prompt_save_file(i32, i32, i32, DeprecatedString const&, DeprecatedString const&, DeprecatedString const&, Core::Stream::OpenMode) override; + virtual void request_file(i32, i32, i32, DeprecatedString const&, Core::File::OpenMode) override; + virtual void prompt_open_file(i32, i32, i32, DeprecatedString const&, DeprecatedString const&, Core::File::OpenMode) override; + virtual void prompt_save_file(i32, i32, i32, DeprecatedString const&, DeprecatedString const&, DeprecatedString const&, Core::File::OpenMode) override; - void prompt_helper(i32, Optional const&, Core::Stream::OpenMode); + void prompt_helper(i32, Optional const&, Core::File::OpenMode); RefPtr create_dummy_child_window(i32, i32); enum class ShouldPrompt { No, Yes }; - void request_file_handler(i32, i32, i32, DeprecatedString const&, Core::Stream::OpenMode, ShouldPrompt); + void request_file_handler(i32, i32, i32, DeprecatedString const&, Core::File::OpenMode, ShouldPrompt); virtual Messages::FileSystemAccessServer::ExposeWindowServerClientIdResponse expose_window_server_client_id() override; - HashMap m_approved_files; + HashMap m_approved_files; }; } diff --git a/Userland/Services/FileSystemAccessServer/FileSystemAccessServer.ipc b/Userland/Services/FileSystemAccessServer/FileSystemAccessServer.ipc index a42379acd1ccc9..a1970b99720e4b 100644 --- a/Userland/Services/FileSystemAccessServer/FileSystemAccessServer.ipc +++ b/Userland/Services/FileSystemAccessServer/FileSystemAccessServer.ipc @@ -4,9 +4,9 @@ endpoint FileSystemAccessServer { request_file_read_only_approved(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString path) =| - request_file(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString path, Core::Stream::OpenMode requested_access) =| - prompt_open_file(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString window_title, DeprecatedString path_to_view, Core::Stream::OpenMode requested_access) =| - prompt_save_file(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString title, DeprecatedString ext, DeprecatedString path_to_view, Core::Stream::OpenMode requested_access) =| + request_file(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString path, Core::File::OpenMode requested_access) =| + prompt_open_file(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString window_title, DeprecatedString path_to_view, Core::File::OpenMode requested_access) =| + prompt_save_file(i32 request_id, i32 window_server_client_id, i32 window_id, DeprecatedString title, DeprecatedString ext, DeprecatedString path_to_view, Core::File::OpenMode requested_access) =| expose_window_server_client_id() => (i32 client_id) } diff --git a/Userland/Services/NetworkServer/main.cpp b/Userland/Services/NetworkServer/main.cpp index 6e4ba5d7a16234..915ff8ea6b8d5c 100644 --- a/Userland/Services/NetworkServer/main.cpp +++ b/Userland/Services/NetworkServer/main.cpp @@ -27,7 +27,7 @@ ErrorOr serenity_main(Main::Arguments) auto config_file = TRY(Core::ConfigFile::open_for_system("Network")); - auto proc_net_adapters_file = TRY(Core::Stream::File::open("/sys/kernel/net/adapters"sv, Core::Stream::OpenMode::Read)); + auto proc_net_adapters_file = TRY(Core::File::open("/sys/kernel/net/adapters"sv, Core::File::OpenMode::Read)); auto data = TRY(proc_net_adapters_file->read_until_eof()); JsonParser parser(data); JsonValue proc_net_adapters_json = TRY(parser.parse()); diff --git a/Userland/Services/RequestServer/GeminiProtocol.cpp b/Userland/Services/RequestServer/GeminiProtocol.cpp index 9412476a9bcc5c..642f24d61308f0 100644 --- a/Userland/Services/RequestServer/GeminiProtocol.cpp +++ b/Userland/Services/RequestServer/GeminiProtocol.cpp @@ -26,7 +26,7 @@ OwnPtr GeminiProtocol::start_request(ConnectionFromClient& client, Depr if (pipe_result.is_error()) return {}; - auto output_stream = MUST(Core::Stream::File::adopt_fd(pipe_result.value().write_fd, Core::Stream::OpenMode::Write)); + auto output_stream = MUST(Core::File::adopt_fd(pipe_result.value().write_fd, Core::File::OpenMode::Write)); auto job = Gemini::Job::construct(request, *output_stream); auto protocol_request = GeminiRequest::create_with_job({}, client, *job, move(output_stream)); protocol_request->set_request_fd(pipe_result.value().read_fd); diff --git a/Userland/Services/RequestServer/GeminiRequest.cpp b/Userland/Services/RequestServer/GeminiRequest.cpp index 048338f30ed3c8..962480f13bd898 100644 --- a/Userland/Services/RequestServer/GeminiRequest.cpp +++ b/Userland/Services/RequestServer/GeminiRequest.cpp @@ -12,7 +12,7 @@ namespace RequestServer { -GeminiRequest::GeminiRequest(ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) +GeminiRequest::GeminiRequest(ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) : Request(client, move(output_stream)) , m_job(move(job)) { @@ -57,7 +57,7 @@ GeminiRequest::~GeminiRequest() m_job->cancel(); } -NonnullOwnPtr GeminiRequest::create_with_job(Badge, ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) +NonnullOwnPtr GeminiRequest::create_with_job(Badge, ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) { return adopt_own(*new GeminiRequest(client, move(job), move(output_stream))); } diff --git a/Userland/Services/RequestServer/GeminiRequest.h b/Userland/Services/RequestServer/GeminiRequest.h index 9039b83980481f..5943f62c8b0cc1 100644 --- a/Userland/Services/RequestServer/GeminiRequest.h +++ b/Userland/Services/RequestServer/GeminiRequest.h @@ -16,14 +16,14 @@ namespace RequestServer { class GeminiRequest final : public Request { public: virtual ~GeminiRequest() override; - static NonnullOwnPtr create_with_job(Badge, ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); + static NonnullOwnPtr create_with_job(Badge, ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); Gemini::Job const& job() const { return *m_job; } virtual URL url() const override { return m_job->url(); } private: - explicit GeminiRequest(ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); + explicit GeminiRequest(ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); virtual void set_certificate(DeprecatedString certificate, DeprecatedString key) override; diff --git a/Userland/Services/RequestServer/HttpCommon.h b/Userland/Services/RequestServer/HttpCommon.h index d54d48ca23b18b..3cc50c7d33c09a 100644 --- a/Userland/Services/RequestServer/HttpCommon.h +++ b/Userland/Services/RequestServer/HttpCommon.h @@ -97,7 +97,7 @@ OwnPtr start_request(TBadgedProtocol&& protocol, ConnectionFromClient& return {}; request.set_body(allocated_body_result.release_value()); - auto output_stream = MUST(Core::Stream::File::adopt_fd(pipe_result.value().write_fd, Core::Stream::OpenMode::Write)); + auto output_stream = MUST(Core::File::adopt_fd(pipe_result.value().write_fd, Core::File::OpenMode::Write)); auto job = TJob::construct(move(request), *output_stream); auto protocol_request = TRequest::create_with_job(forward(protocol), client, (TJob&)*job, move(output_stream)); protocol_request->set_request_fd(pipe_result.value().read_fd); diff --git a/Userland/Services/RequestServer/HttpRequest.cpp b/Userland/Services/RequestServer/HttpRequest.cpp index 861c7a93b36e8b..0249650c3d97ac 100644 --- a/Userland/Services/RequestServer/HttpRequest.cpp +++ b/Userland/Services/RequestServer/HttpRequest.cpp @@ -11,7 +11,7 @@ namespace RequestServer { -HttpRequest::HttpRequest(ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) +HttpRequest::HttpRequest(ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) : Request(client, move(output_stream)) , m_job(job) { @@ -25,7 +25,7 @@ HttpRequest::~HttpRequest() m_job->cancel(); } -NonnullOwnPtr HttpRequest::create_with_job(Badge&&, ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) +NonnullOwnPtr HttpRequest::create_with_job(Badge&&, ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) { return adopt_own(*new HttpRequest(client, move(job), move(output_stream))); } diff --git a/Userland/Services/RequestServer/HttpRequest.h b/Userland/Services/RequestServer/HttpRequest.h index 624cccf4dc1d63..6833e74d6c559a 100644 --- a/Userland/Services/RequestServer/HttpRequest.h +++ b/Userland/Services/RequestServer/HttpRequest.h @@ -17,7 +17,7 @@ namespace RequestServer { class HttpRequest final : public Request { public: virtual ~HttpRequest() override; - static NonnullOwnPtr create_with_job(Badge&&, ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); + static NonnullOwnPtr create_with_job(Badge&&, ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); HTTP::Job& job() { return m_job; } HTTP::Job const& job() const { return m_job; } @@ -25,7 +25,7 @@ class HttpRequest final : public Request { virtual URL url() const override { return m_job->url(); } private: - explicit HttpRequest(ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); + explicit HttpRequest(ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); NonnullRefPtr m_job; }; diff --git a/Userland/Services/RequestServer/HttpsRequest.cpp b/Userland/Services/RequestServer/HttpsRequest.cpp index dcf2c4da71aeb2..f9c9ba0272cfa0 100644 --- a/Userland/Services/RequestServer/HttpsRequest.cpp +++ b/Userland/Services/RequestServer/HttpsRequest.cpp @@ -11,7 +11,7 @@ namespace RequestServer { -HttpsRequest::HttpsRequest(ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) +HttpsRequest::HttpsRequest(ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) : Request(client, move(output_stream)) , m_job(job) { @@ -30,7 +30,7 @@ HttpsRequest::~HttpsRequest() m_job->cancel(); } -NonnullOwnPtr HttpsRequest::create_with_job(Badge&&, ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) +NonnullOwnPtr HttpsRequest::create_with_job(Badge&&, ConnectionFromClient& client, NonnullRefPtr job, NonnullOwnPtr&& output_stream) { return adopt_own(*new HttpsRequest(client, move(job), move(output_stream))); } diff --git a/Userland/Services/RequestServer/HttpsRequest.h b/Userland/Services/RequestServer/HttpsRequest.h index 1add0e1cee0bcc..01eb3870cd98df 100644 --- a/Userland/Services/RequestServer/HttpsRequest.h +++ b/Userland/Services/RequestServer/HttpsRequest.h @@ -16,7 +16,7 @@ namespace RequestServer { class HttpsRequest final : public Request { public: virtual ~HttpsRequest() override; - static NonnullOwnPtr create_with_job(Badge&&, ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); + static NonnullOwnPtr create_with_job(Badge&&, ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); HTTP::HttpsJob& job() { return m_job; } HTTP::HttpsJob const& job() const { return m_job; } @@ -24,7 +24,7 @@ class HttpsRequest final : public Request { virtual URL url() const override { return m_job->url(); } private: - explicit HttpsRequest(ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); + explicit HttpsRequest(ConnectionFromClient&, NonnullRefPtr, NonnullOwnPtr&&); virtual void set_certificate(DeprecatedString certificate, DeprecatedString key) override; diff --git a/Userland/Services/RequestServer/Request.cpp b/Userland/Services/RequestServer/Request.cpp index b1d00d8e03766d..d766bd85087a7c 100644 --- a/Userland/Services/RequestServer/Request.cpp +++ b/Userland/Services/RequestServer/Request.cpp @@ -12,7 +12,7 @@ namespace RequestServer { // FIXME: What about rollover? static i32 s_next_id = 1; -Request::Request(ConnectionFromClient& client, NonnullOwnPtr&& output_stream) +Request::Request(ConnectionFromClient& client, NonnullOwnPtr&& output_stream) : m_client(client) , m_id(s_next_id++) , m_output_stream(move(output_stream)) diff --git a/Userland/Services/RequestServer/Request.h b/Userland/Services/RequestServer/Request.h index 6b4a9a107a032f..7633032e0797c8 100644 --- a/Userland/Services/RequestServer/Request.h +++ b/Userland/Services/RequestServer/Request.h @@ -40,10 +40,10 @@ class Request { void did_request_certificates(); void set_response_headers(HashMap const&); void set_downloaded_size(size_t size) { m_downloaded_size = size; } - Core::Stream::File const& output_stream() const { return *m_output_stream; } + Core::File const& output_stream() const { return *m_output_stream; } protected: - explicit Request(ConnectionFromClient&, NonnullOwnPtr&&); + explicit Request(ConnectionFromClient&, NonnullOwnPtr&&); private: ConnectionFromClient& m_client; @@ -52,7 +52,7 @@ class Request { Optional m_status_code; Optional m_total_size {}; size_t m_downloaded_size { 0 }; - NonnullOwnPtr m_output_stream; + NonnullOwnPtr m_output_stream; HashMap m_response_headers; }; diff --git a/Userland/Services/WebServer/Client.cpp b/Userland/Services/WebServer/Client.cpp index ce90bdff07a479..ae65737e3f42e2 100644 --- a/Userland/Services/WebServer/Client.cpp +++ b/Userland/Services/WebServer/Client.cpp @@ -16,6 +16,7 @@ #include #include #include +#include #include #include #include @@ -165,7 +166,7 @@ ErrorOr Client::handle_request(ReadonlyBytes raw_request) return false; } - auto stream = TRY(Core::Stream::File::open(real_path.bytes_as_string_view(), Core::Stream::OpenMode::Read)); + auto stream = TRY(Core::File::open(real_path.bytes_as_string_view(), Core::File::OpenMode::Read)); auto const info = ContentInfo { .type = TRY(String::from_deprecated_string(Core::guess_mime_type_based_on_filename(real_path.bytes_as_string_view()))), diff --git a/Userland/Utilities/arp.cpp b/Userland/Utilities/arp.cpp index 066474b46be769..89324c8f7ab6c1 100644 --- a/Userland/Utilities/arp.cpp +++ b/Userland/Utilities/arp.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -92,7 +93,7 @@ ErrorOr serenity_main(Main::Arguments arguments) outln(); if (!flag_set && !flag_delete) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/net/arp"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/net/arp"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); diff --git a/Userland/Utilities/base64.cpp b/Userland/Utilities/base64.cpp index a622ea8ca6e3ca..9aa05d3197557a 100644 --- a/Userland/Utilities/base64.cpp +++ b/Userland/Utilities/base64.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -22,7 +23,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(filepath, "", "file", Core::ArgsParser::Required::No); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(filepath, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(filepath, Core::File::OpenMode::Read)); ByteBuffer buffer = TRY(file->read_until_eof()); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/cat.cpp b/Userland/Utilities/cat.cpp index 000b26d73940bd..ea5e558f0ee595 100644 --- a/Userland/Utilities/cat.cpp +++ b/Userland/Utilities/cat.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include #include @@ -25,11 +26,11 @@ ErrorOr serenity_main(Main::Arguments arguments) if (paths.is_empty()) paths.append("-"sv); - Vector> files; + Vector> files; TRY(files.try_ensure_capacity(paths.size())); for (auto const& path : paths) { - if (auto result = Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read); result.is_error()) + if (auto result = Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read); result.is_error()) warnln("Failed to open {}: {}", path, result.release_error()); else files.unchecked_append(result.release_value()); diff --git a/Userland/Utilities/checksum.cpp b/Userland/Utilities/checksum.cpp index 5ac56e8cb47b3d..abb566ab043288 100644 --- a/Userland/Utilities/checksum.cpp +++ b/Userland/Utilities/checksum.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -55,7 +56,7 @@ ErrorOr serenity_main(Main::Arguments arguments) int failed_verification_count = 0; for (auto const& path : paths) { - auto file_or_error = Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read); if (file_or_error.is_error()) { ++read_fail_count; has_error = true; @@ -87,7 +88,7 @@ ErrorOr serenity_main(Main::Arguments arguments) // line[0] = checksum // line[1] = filename StringView const filename = line[1]; - auto file_from_filename_or_error = Core::Stream::File::open_file_or_standard_stream(filename, Core::Stream::OpenMode::Read); + auto file_from_filename_or_error = Core::File::open_file_or_standard_stream(filename, Core::File::OpenMode::Read); if (file_from_filename_or_error.is_error()) { ++read_fail_count; warnln("{}: {}", filename, file_from_filename_or_error.release_error()); diff --git a/Userland/Utilities/cksum.cpp b/Userland/Utilities/cksum.cpp index 08c7cb0be38a71..ba1b1028019259 100644 --- a/Userland/Utilities/cksum.cpp +++ b/Userland/Utilities/cksum.cpp @@ -5,6 +5,7 @@ */ #include +#include #include #include #include @@ -46,7 +47,7 @@ ErrorOr serenity_main(Main::Arguments arguments) Array buffer; for (auto& path : paths) { - auto file_or_error = Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read); auto filepath = (path == "-") ? "/dev/stdin" : path; if (file_or_error.is_error()) { warnln("{}: {}: {}", arguments.strings[0], filepath, file_or_error.error()); diff --git a/Userland/Utilities/cmp.cpp b/Userland/Utilities/cmp.cpp index 883709899277fc..ac213cb88bd549 100644 --- a/Userland/Utilities/cmp.cpp +++ b/Userland/Utilities/cmp.cpp @@ -5,20 +5,21 @@ */ #include +#include #include #include #include #include -static ErrorOr> open_file_or_stdin(DeprecatedString const& filename) +static ErrorOr> open_file_or_stdin(DeprecatedString const& filename) { - OwnPtr file; + OwnPtr file; if (filename == "-") { - file = TRY(Core::Stream::File::adopt_fd(STDIN_FILENO, Core::Stream::OpenMode::Read)); + file = TRY(Core::File::adopt_fd(STDIN_FILENO, Core::File::OpenMode::Read)); } else { - file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Read)); + file = TRY(Core::File::open(filename, Core::File::OpenMode::Read)); } - return TRY(Core::Stream::BufferedFile::create(file.release_nonnull())); + return TRY(Core::BufferedFile::create(file.release_nonnull())); } ErrorOr serenity_main(Main::Arguments arguments) diff --git a/Userland/Utilities/comm.cpp b/Userland/Utilities/comm.cpp index 370bdc4ea1085b..0040c94ce42060 100644 --- a/Userland/Utilities/comm.cpp +++ b/Userland/Utilities/comm.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include #include @@ -62,7 +63,7 @@ ErrorOr serenity_main(Main::Arguments arguments) } auto open_file = [](DeprecatedString const& path, auto& file, int file_number) { - auto file_or_error = Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read); if (file_or_error.is_error()) { warnln("Failed to open file{} '{}': {}", file_number, path, file_or_error.error()); return false; @@ -73,7 +74,7 @@ ErrorOr serenity_main(Main::Arguments arguments) return false; } - auto buffered_file_or_error = Core::Stream::BufferedFile::create(file_or_error.release_value()); + auto buffered_file_or_error = Core::BufferedFile::create(file_or_error.release_value()); if (buffered_file_or_error.is_error()) { warnln("Failed to create buffer for file{} '{}': {}", file_number, path, buffered_file_or_error.error()); return false; @@ -83,8 +84,8 @@ ErrorOr serenity_main(Main::Arguments arguments) return true; }; - OwnPtr file1; - OwnPtr file2; + OwnPtr file1; + OwnPtr file2; if (!(open_file(file1_path, file1, 1) && open_file(file2_path, file2, 2))) return 1; diff --git a/Userland/Utilities/copy.cpp b/Userland/Utilities/copy.cpp index b3103c7146a1c7..c9122d46f3ef4e 100644 --- a/Userland/Utilities/copy.cpp +++ b/Userland/Utilities/copy.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -42,7 +43,7 @@ static ErrorOr parse_options(Main::Arguments arguments) // We're not copying anything. } else if (text.is_empty()) { // Copy our stdin. - auto c_stdin = TRY(Core::Stream::File::standard_input()); + auto c_stdin = TRY(Core::File::standard_input()); auto buffer = TRY(c_stdin->read_until_eof()); dbgln("Read size {}", buffer.size()); dbgln("Read data: `{}`", StringView(buffer.bytes())); diff --git a/Userland/Utilities/cpp-lexer.cpp b/Userland/Utilities/cpp-lexer.cpp index fdbe49c5e8ad01..aa0fdafe20bb19 100644 --- a/Userland/Utilities/cpp-lexer.cpp +++ b/Userland/Utilities/cpp-lexer.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -17,7 +18,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(path, "Cpp File", "cpp-file", Core::ArgsParser::Required::Yes); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto content = TRY(file->read_until_eof()); StringView content_view(content); diff --git a/Userland/Utilities/cpp-parser.cpp b/Userland/Utilities/cpp-parser.cpp index 91e6f5a4de8f8c..683b9ed241a2c2 100644 --- a/Userland/Utilities/cpp-parser.cpp +++ b/Userland/Utilities/cpp-parser.cpp @@ -5,6 +5,7 @@ */ #include +#include #include #include #include @@ -20,7 +21,7 @@ ErrorOr serenity_main(Main::Arguments arguments) if (path.is_empty()) path = "Source/little/main.cpp"sv; - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto content = TRY(file->read_until_eof()); StringView content_view(content); diff --git a/Userland/Utilities/cpp-preprocessor.cpp b/Userland/Utilities/cpp-preprocessor.cpp index 34e6148471f20e..4414e53752d0b1 100644 --- a/Userland/Utilities/cpp-preprocessor.cpp +++ b/Userland/Utilities/cpp-preprocessor.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -19,7 +20,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(print_definitions, "Print preprocessor definitions", "definitions", 'D'); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto content = TRY(file->read_until_eof()); DeprecatedString name = LexicalPath::basename(path); Cpp::Preprocessor cpp(name, StringView { content }); diff --git a/Userland/Utilities/df.cpp b/Userland/Utilities/df.cpp index a80945aa81d8a1..56bef8ed890614 100644 --- a/Userland/Utilities/df.cpp +++ b/Userland/Utilities/df.cpp @@ -8,6 +8,7 @@ #include #include #include +#include #include #include #include @@ -36,7 +37,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(flag_inode_info, "Show inode information as well", "inodes", 'i'); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open("/sys/kernel/df"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/df"sv, Core::File::OpenMode::Read)); Vector headers; TRY(headers.try_append(flag_human_readable ? "Size"sv : "Blocks"sv)); diff --git a/Userland/Utilities/diff.cpp b/Userland/Utilities/diff.cpp index 9ec39cabe1ad9a..c26318ac400ac4 100644 --- a/Userland/Utilities/diff.cpp +++ b/Userland/Utilities/diff.cpp @@ -5,6 +5,7 @@ */ #include +#include #include #include #include @@ -23,8 +24,8 @@ ErrorOr serenity_main(Main::Arguments arguments) parser.add_positional_argument(filename2, "Second file to compare", "file2", Core::ArgsParser::Required::Yes); parser.parse(arguments); - auto file1 = TRY(Core::Stream::File::open(filename1, Core::Stream::OpenMode::Read)); - auto file2 = TRY(Core::Stream::File::open(filename2, Core::Stream::OpenMode::Read)); + auto file1 = TRY(Core::File::open(filename1, Core::File::OpenMode::Read)); + auto file2 = TRY(Core::File::open(filename2, Core::File::OpenMode::Read)); bool color_output = TRY(Core::System::isatty(STDOUT_FILENO)); diff --git a/Userland/Utilities/dmesg.cpp b/Userland/Utilities/dmesg.cpp index cb5e94baf45309..459b06e4259542 100644 --- a/Userland/Utilities/dmesg.cpp +++ b/Userland/Utilities/dmesg.cpp @@ -4,6 +4,7 @@ * SPDX-License-Identifier: BSD-2-Clause */ +#include #include #include #include @@ -14,7 +15,7 @@ ErrorOr serenity_main(Main::Arguments) TRY(Core::System::unveil("/sys/kernel/dmesg", "r")); TRY(Core::System::unveil(nullptr, nullptr)); - auto file = TRY(Core::Stream::File::open("/sys/kernel/dmesg"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/dmesg"sv, Core::File::OpenMode::Read)); auto buffer = TRY(file->read_until_eof()); out("{}", StringView { buffer }); return 0; diff --git a/Userland/Utilities/du.cpp b/Userland/Utilities/du.cpp index 5de11b6db43c8b..041ef2b4d5c339 100644 --- a/Userland/Utilities/du.cpp +++ b/Userland/Utilities/du.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -116,7 +117,7 @@ ErrorOr parse_args(Main::Arguments arguments, Vector& fi if (!pattern.is_empty()) du_option.excluded_patterns.append(pattern); if (!exclude_from.is_empty()) { - auto file = TRY(Core::Stream::File::open(exclude_from, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(exclude_from, Core::File::OpenMode::Read)); auto const buff = TRY(file->read_until_eof()); if (!buff.is_empty()) { DeprecatedString patterns = DeprecatedString::copy(buff, Chomp); diff --git a/Userland/Utilities/file.cpp b/Userland/Utilities/file.cpp index da25e832ab2548..ecf71bb53221b7 100644 --- a/Userland/Utilities/file.cpp +++ b/Userland/Utilities/file.cpp @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -158,7 +159,7 @@ ErrorOr serenity_main(Main::Arguments arguments) auto buffer = TRY(ByteBuffer::create_uninitialized(0x9006)); for (auto const& path : paths) { - auto file_or_error = Core::Stream::File::open(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(path, Core::File::OpenMode::Read); if (file_or_error.is_error()) { perror(path.characters()); all_ok = false; diff --git a/Userland/Utilities/fortune.cpp b/Userland/Utilities/fortune.cpp index c641d3e4a0a98c..f6da0e96311226 100644 --- a/Userland/Utilities/fortune.cpp +++ b/Userland/Utilities/fortune.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -81,7 +82,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(path, "Path to JSON file with quotes (/res/fortunes.json by default)", "path", Core::ArgsParser::Required::No); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); TRY(Core::System::unveil("/etc/timezone", "r")); TRY(Core::System::unveil(nullptr, nullptr)); diff --git a/Userland/Utilities/gml-format.cpp b/Userland/Utilities/gml-format.cpp index a332fb305127dc..ac4a7cbcbb73c0 100644 --- a/Userland/Utilities/gml-format.cpp +++ b/Userland/Utilities/gml-format.cpp @@ -5,6 +5,7 @@ */ #include +#include #include #include #include @@ -13,8 +14,8 @@ static ErrorOr format_file(StringView path, bool inplace) { auto read_from_stdin = path == "-"; - auto open_mode = (inplace && !read_from_stdin) ? Core::Stream::OpenMode::ReadWrite : Core::Stream::OpenMode::Read; - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, open_mode)); + auto open_mode = (inplace && !read_from_stdin) ? Core::File::OpenMode::ReadWrite : Core::File::OpenMode::Read; + auto file = TRY(Core::File::open_file_or_standard_stream(path, open_mode)); auto contents = TRY(file->read_until_eof()); auto formatted_gml_or_error = GUI::GML::format_gml(contents); diff --git a/Userland/Utilities/grep.cpp b/Userland/Utilities/grep.cpp index 9004e841f695e4..31d8e0079423c4 100644 --- a/Userland/Utilities/grep.cpp +++ b/Userland/Utilities/grep.cpp @@ -13,6 +13,7 @@ #include #include #include +#include #include #include #include @@ -215,8 +216,8 @@ ErrorOr serenity_main(Main::Arguments args) auto handle_file = [&matches, binary_mode, count_lines, quiet_mode, user_specified_multiple_files, &matched_line_count, &did_match_something](StringView filename, bool print_filename) -> ErrorOr { - auto file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Read)); - auto buffered_file = TRY(Core::Stream::BufferedFile::create(move(file))); + auto file = TRY(Core::File::open(filename, Core::File::OpenMode::Read)); + auto buffered_file = TRY(Core::BufferedFile::create(move(file))); for (size_t line_number = 1; TRY(buffered_file->can_read_line()); ++line_number) { Array buffer; diff --git a/Userland/Utilities/gron.cpp b/Userland/Utilities/gron.cpp index 925cc70fea213e..34721bec35a2be 100644 --- a/Userland/Utilities/gron.cpp +++ b/Userland/Utilities/gron.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -54,7 +55,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(path, "Input", "input", Core::ArgsParser::Required::No); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/gunzip.cpp b/Userland/Utilities/gunzip.cpp index 1c5b395b0f95a4..531c753e3fc064 100644 --- a/Userland/Utilities/gunzip.cpp +++ b/Userland/Utilities/gunzip.cpp @@ -6,11 +6,12 @@ #include #include +#include #include #include #include -static ErrorOr decompress_file(NonnullOwnPtr input_stream, AK::Stream& output_stream) +static ErrorOr decompress_file(NonnullOwnPtr input_stream, AK::Stream& output_stream) { auto gzip_stream = Compress::GzipDecompressor { move(input_stream) }; @@ -50,8 +51,8 @@ ErrorOr serenity_main(Main::Arguments args) output_filename = filename; } - auto input_stream_result = TRY(Core::Stream::File::open(input_filename, Core::Stream::OpenMode::Read)); - auto output_stream = write_to_stdout ? TRY(Core::Stream::File::standard_output()) : TRY(Core::Stream::File::open(output_filename, Core::Stream::OpenMode::Write)); + auto input_stream_result = TRY(Core::File::open(input_filename, Core::File::OpenMode::Read)); + auto output_stream = write_to_stdout ? TRY(Core::File::standard_output()) : TRY(Core::File::open(output_filename, Core::File::OpenMode::Write)); TRY(decompress_file(move(input_stream_result), *output_stream)); diff --git a/Userland/Utilities/gzip.cpp b/Userland/Utilities/gzip.cpp index 6f5aa5e230f4bc..28fbdc38c20ab5 100644 --- a/Userland/Utilities/gzip.cpp +++ b/Userland/Utilities/gzip.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -55,7 +56,7 @@ ErrorOr serenity_main(Main::Arguments arguments) else output_bytes = TRY(Compress::GzipCompressor::compress_all(input_bytes)); - auto output_stream = write_to_stdout ? TRY(Core::Stream::File::standard_output()) : TRY(Core::Stream::File::open(output_filename, Core::Stream::OpenMode::Write)); + auto output_stream = write_to_stdout ? TRY(Core::File::standard_output()) : TRY(Core::File::open(output_filename, Core::File::OpenMode::Write)); TRY(output_stream->write_entire_buffer(output_bytes)); if (!keep_input_files) { diff --git a/Userland/Utilities/headless-browser.cpp b/Userland/Utilities/headless-browser.cpp index c24754b3d88406..43e2394e5efac2 100644 --- a/Userland/Utilities/headless-browser.cpp +++ b/Userland/Utilities/headless-browser.cpp @@ -694,7 +694,7 @@ static void load_page_for_screenshot_and_exit(HeadlessBrowserPageClient& page_cl if (Core::DeprecatedFile::exists(output_file_path)) MUST(Core::DeprecatedFile::remove(output_file_path, Core::DeprecatedFile::RecursionMode::Disallowed)); - auto output_file = MUST(Core::Stream::File::open(output_file_path, Core::Stream::OpenMode::Write)); + auto output_file = MUST(Core::File::open(output_file_path, Core::File::OpenMode::Write)); auto output_rect = page_client.screen_rect(); auto output_bitmap = MUST(Gfx::Bitmap::create(Gfx::BitmapFormat::BGRx8888, output_rect.size().to_type())); diff --git a/Userland/Utilities/hexdump.cpp b/Userland/Utilities/hexdump.cpp index 00fe33f4e1039e..008c46e2dc883c 100644 --- a/Userland/Utilities/hexdump.cpp +++ b/Userland/Utilities/hexdump.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include #include @@ -37,7 +38,7 @@ ErrorOr serenity_main(Main::Arguments args) args_parser.add_option(seek_to, "Seek to a byte offset", "seek", 's', "offset"); args_parser.parse(args); - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read)); if (seek_to.has_value()) TRY(file->seek(seek_to.value(), SeekMode::SetPosition)); diff --git a/Userland/Utilities/ifconfig.cpp b/Userland/Utilities/ifconfig.cpp index 5b8dc637a3f208..ad259079443438 100644 --- a/Userland/Utilities/ifconfig.cpp +++ b/Userland/Utilities/ifconfig.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -31,7 +32,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.parse(arguments); if (value_ipv4.is_empty() && value_adapter.is_empty() && value_mask.is_empty()) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/net/adapters"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/net/adapters"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); diff --git a/Userland/Utilities/js.cpp b/Userland/Utilities/js.cpp index 3ca9af8d67a49d..2528eb7b788597 100644 --- a/Userland/Utilities/js.cpp +++ b/Userland/Utilities/js.cpp @@ -95,7 +95,7 @@ enum class PrintTarget { static ErrorOr print(JS::Value value, PrintTarget target = PrintTarget::StandardOutput) { - auto stream = TRY(target == PrintTarget::StandardError ? Core::Stream::File::standard_error() : Core::Stream::File::standard_output()); + auto stream = TRY(target == PrintTarget::StandardError ? Core::File::standard_error() : Core::File::standard_output()); return print(value, *stream); } @@ -185,7 +185,7 @@ static ErrorOr read_next_piece() static ErrorOr write_to_file(String const& path) { - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Write, 0666)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Write, 0666)); for (size_t i = 0; i < g_repl_statements.size(); i++) { auto line = g_repl_statements[i].bytes(); if (line.size() > 0 && i != g_repl_statements.size() - 1) { @@ -334,7 +334,7 @@ static JS::ThrowCompletionOr load_ini_impl(JS::VM& vm) auto& realm = *vm.current_realm(); auto filename = TRY(vm.argument(0).to_deprecated_string(vm)); - auto file_or_error = Core::Stream::File::open(filename, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(filename, Core::File::OpenMode::Read); if (file_or_error.is_error()) return vm.throw_completion(DeprecatedString::formatted("Failed to open '{}': {}", filename, file_or_error.error())); @@ -354,7 +354,7 @@ static JS::ThrowCompletionOr load_ini_impl(JS::VM& vm) static JS::ThrowCompletionOr load_json_impl(JS::VM& vm) { auto filename = TRY(vm.argument(0).to_string(vm)); - auto file_or_error = Core::Stream::File::open(filename, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(filename, Core::File::OpenMode::Read); if (file_or_error.is_error()) return vm.throw_completion(DeprecatedString::formatted("Failed to open '{}': {}", filename, file_or_error.error())); @@ -871,7 +871,7 @@ ErrorOr serenity_main(Main::Arguments arguments) warnln("Warning: Multiple files supplied, this will concatenate the sources and resolve modules as if it was the first file"); for (auto& path : script_paths) { - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto source = StringView { file_contents }; diff --git a/Userland/Utilities/json.cpp b/Userland/Utilities/json.cpp index d0653dc453eecb..f8ce64e7bf966c 100644 --- a/Userland/Utilities/json.cpp +++ b/Userland/Utilities/json.cpp @@ -14,6 +14,7 @@ #include #include #include +#include #include #include #include @@ -43,7 +44,7 @@ ErrorOr serenity_main(Main::Arguments arguments) VERIFY(spaces_in_indent >= 0); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/lsblk.cpp b/Userland/Utilities/lsblk.cpp index e6d611ff458be0..443624b420c7e9 100644 --- a/Userland/Utilities/lsblk.cpp +++ b/Userland/Utilities/lsblk.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -38,19 +39,19 @@ ErrorOr serenity_main(Main::Arguments arguments) while (di.has_next()) { auto dir = di.next_path(); auto command_set_filename = DeprecatedString::formatted("/sys/devices/storage/{}/command_set", dir); - auto command_set_file = Core::Stream::File::open(command_set_filename, Core::Stream::OpenMode::Read); + auto command_set_file = Core::File::open(command_set_filename, Core::File::OpenMode::Read); if (command_set_file.is_error()) { dbgln("Error: Could not open {}: {}", command_set_filename, command_set_file.error()); continue; } auto last_lba_filename = DeprecatedString::formatted("/sys/devices/storage/{}/last_lba", dir); - auto last_lba_file = Core::Stream::File::open(last_lba_filename, Core::Stream::OpenMode::Read); + auto last_lba_file = Core::File::open(last_lba_filename, Core::File::OpenMode::Read); if (last_lba_file.is_error()) { dbgln("Error: Could not open {}: {}", last_lba_filename, last_lba_file.error()); continue; } auto sector_size_filename = DeprecatedString::formatted("/sys/devices/storage/{}/sector_size", dir); - auto sector_size_file = Core::Stream::File::open(sector_size_filename, Core::Stream::OpenMode::Read); + auto sector_size_file = Core::File::open(sector_size_filename, Core::File::OpenMode::Read); if (sector_size_file.is_error()) { dbgln("Error: Could not open {}: {}", sector_size_filename, sector_size_file.error()); continue; diff --git a/Userland/Utilities/lscpu.cpp b/Userland/Utilities/lscpu.cpp index 4a9f143c788685..21b00cc4921f4c 100644 --- a/Userland/Utilities/lscpu.cpp +++ b/Userland/Utilities/lscpu.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -59,7 +60,7 @@ ErrorOr serenity_main(Main::Arguments) TRY(Core::System::unveil("/sys/kernel/cpuinfo", "r")); TRY(Core::System::unveil(nullptr, nullptr)); - auto file = TRY(Core::Stream::File::open("/sys/kernel/cpuinfo"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/cpuinfo"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); auto const& array = json.as_array(); diff --git a/Userland/Utilities/lsirq.cpp b/Userland/Utilities/lsirq.cpp index b572563905c972..55dc1c34b0e128 100644 --- a/Userland/Utilities/lsirq.cpp +++ b/Userland/Utilities/lsirq.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include #include @@ -17,7 +18,7 @@ ErrorOr serenity_main(Main::Arguments) TRY(Core::System::unveil("/sys/kernel/interrupts", "r")); TRY(Core::System::unveil(nullptr, nullptr)); - auto proc_interrupts = TRY(Core::Stream::File::open("/sys/kernel/interrupts"sv, Core::Stream::OpenMode::Read)); + auto proc_interrupts = TRY(Core::File::open("/sys/kernel/interrupts"sv, Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/lsjails.cpp b/Userland/Utilities/lsjails.cpp index 7f0fb392cbe218..06b6d02fd33aac 100644 --- a/Userland/Utilities/lsjails.cpp +++ b/Userland/Utilities/lsjails.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -16,7 +17,7 @@ ErrorOr serenity_main(Main::Arguments) TRY(Core::System::unveil("/sys/kernel/jails", "r")); TRY(Core::System::unveil(nullptr, nullptr)); - auto jails_data = TRY(Core::Stream::File::open("/sys/kernel/jails"sv, Core::Stream::OpenMode::Read)); + auto jails_data = TRY(Core::File::open("/sys/kernel/jails"sv, Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/lsof.cpp b/Userland/Utilities/lsof.cpp index c59d5922cbac54..cc7901fab8a3b9 100644 --- a/Userland/Utilities/lsof.cpp +++ b/Userland/Utilities/lsof.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -65,7 +66,7 @@ static bool parse_name(StringView name, OpenFile& file) static Vector get_open_files_by_pid(pid_t pid) { - auto file = Core::Stream::File::open(DeprecatedString::formatted("/proc/{}/fds", pid), Core::Stream::OpenMode::Read); + auto file = Core::File::open(DeprecatedString::formatted("/proc/{}/fds", pid), Core::File::OpenMode::Read); if (file.is_error()) { outln("lsof: PID {}: {}", pid, file.error()); return Vector(); diff --git a/Userland/Utilities/lspci.cpp b/Userland/Utilities/lspci.cpp index feac6a4ae64b0a..6f97cf5414630f 100644 --- a/Userland/Utilities/lspci.cpp +++ b/Userland/Utilities/lspci.cpp @@ -13,6 +13,7 @@ #include #include #include +#include #include #include #include @@ -89,31 +90,31 @@ ErrorOr serenity_main(Main::Arguments arguments) auto function = convert_sysfs_value_to_uint(function_parts[1]); auto vendor_id_filename = DeprecatedString::formatted("/sys/bus/pci/{}/vendor", dir); - auto vendor_id_file = Core::Stream::File::open(vendor_id_filename, Core::Stream::OpenMode::Read); + auto vendor_id_file = Core::File::open(vendor_id_filename, Core::File::OpenMode::Read); if (vendor_id_file.is_error()) { dbgln("Error: Could not open {}: {}", vendor_id_filename, vendor_id_file.error()); continue; } auto device_id_filename = DeprecatedString::formatted("/sys/bus/pci/{}/device_id", dir); - auto device_id_file = Core::Stream::File::open(device_id_filename, Core::Stream::OpenMode::Read); + auto device_id_file = Core::File::open(device_id_filename, Core::File::OpenMode::Read); if (device_id_file.is_error()) { dbgln("Error: Could not open {}: {}", device_id_filename, device_id_file.error()); continue; } auto class_id_filename = DeprecatedString::formatted("/sys/bus/pci/{}/class", dir); - auto class_id_file = Core::Stream::File::open(class_id_filename, Core::Stream::OpenMode::Read); + auto class_id_file = Core::File::open(class_id_filename, Core::File::OpenMode::Read); if (class_id_file.is_error()) { dbgln("Error: Could not open {}: {}", class_id_filename, class_id_file.error()); continue; } auto subclass_id_filename = DeprecatedString::formatted("/sys/bus/pci/{}/subclass", dir); - auto subclass_id_file = Core::Stream::File::open(subclass_id_filename, Core::Stream::OpenMode::Read); + auto subclass_id_file = Core::File::open(subclass_id_filename, Core::File::OpenMode::Read); if (subclass_id_file.is_error()) { dbgln("Error: Could not open {}: {}", subclass_id_filename, subclass_id_file.error()); continue; } auto revision_id_filename = DeprecatedString::formatted("/sys/bus/pci/{}/revision", dir); - auto revision_id_file = Core::Stream::File::open(revision_id_filename, Core::Stream::OpenMode::Read); + auto revision_id_file = Core::File::open(revision_id_filename, Core::File::OpenMode::Read); if (revision_id_file.is_error()) { dbgln("Error: Could not open {}: {}", revision_id_filename, revision_id_file.error()); continue; @@ -177,7 +178,7 @@ ErrorOr serenity_main(Main::Arguments arguments) continue; for (size_t bar_index = 0; bar_index <= 5; bar_index++) { auto bar_value_filename = DeprecatedString::formatted("/sys/bus/pci/{}/bar{}", dir, bar_index); - auto bar_value_file = Core::Stream::File::open(bar_value_filename, Core::Stream::OpenMode::Read); + auto bar_value_file = Core::File::open(bar_value_filename, Core::File::OpenMode::Read); if (bar_value_file.is_error()) { dbgln("Error: Could not open {}: {}", bar_value_filename, bar_value_file.error()); continue; diff --git a/Userland/Utilities/lsusb.cpp b/Userland/Utilities/lsusb.cpp index 5f4806b96c73b1..a3eeeb879645b6 100644 --- a/Userland/Utilities/lsusb.cpp +++ b/Userland/Utilities/lsusb.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -48,7 +49,7 @@ ErrorOr serenity_main(Main::Arguments arguments) while (usb_devices.has_next()) { auto full_path = LexicalPath(usb_devices.next_full_path()); - auto proc_usb_device = Core::Stream::File::open(full_path.string(), Core::Stream::OpenMode::Read); + auto proc_usb_device = Core::File::open(full_path.string(), Core::File::OpenMode::Read); if (proc_usb_device.is_error()) { warnln("Failed to open {}: {}", full_path.string(), proc_usb_device.error()); continue; diff --git a/Userland/Utilities/man.cpp b/Userland/Utilities/man.cpp index 80546b0e6bd969..95d365019cdef3 100644 --- a/Userland/Utilities/man.cpp +++ b/Userland/Utilities/man.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -84,7 +85,7 @@ ErrorOr serenity_main(Main::Arguments arguments) .to_deprecated_string(); pid_t pager_pid = TRY(pipe_to_pager(pager)); - auto file = TRY(Core::Stream::File::open(TRY(page->path()), Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(TRY(page->path()), Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio proc")); diff --git a/Userland/Utilities/markdown-check.cpp b/Userland/Utilities/markdown-check.cpp index 55c45d2d75ef1b..549a18f272dea8 100644 --- a/Userland/Utilities/markdown-check.cpp +++ b/Userland/Utilities/markdown-check.cpp @@ -20,6 +20,7 @@ #include #include #include +#include #include #include #include @@ -249,7 +250,7 @@ ErrorOr serenity_main(Main::Arguments arguments) outln("Reading and parsing Markdown files ..."); HashMap files; for (auto path : file_paths) { - auto file_or_error = Core::Stream::File::open(path, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(path, Core::File::OpenMode::Read); if (file_or_error.is_error()) { warnln("Failed to open {}: {}", path, file_or_error.error()); // Since this should never happen anyway, fail early. diff --git a/Userland/Utilities/md.cpp b/Userland/Utilities/md.cpp index a027043219408a..6c28fc31f9c74f 100644 --- a/Userland/Utilities/md.cpp +++ b/Userland/Utilities/md.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -41,7 +42,7 @@ ErrorOr serenity_main(Main::Arguments arguments) } } - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(filename, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(filename, Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/mount.cpp b/Userland/Utilities/mount.cpp index e1d91b41466d07..3b6f9f323d5027 100644 --- a/Userland/Utilities/mount.cpp +++ b/Userland/Utilities/mount.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -112,8 +113,8 @@ static ErrorOr mount_all() bool all_ok = true; auto process_fstab_entries = [&](StringView path) -> ErrorOr { - auto file_unbuffered = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); - auto file = TRY(Core::Stream::BufferedFile::create(move(file_unbuffered))); + auto file_unbuffered = TRY(Core::File::open(path, Core::File::OpenMode::Read)); + auto file = TRY(Core::BufferedFile::create(move(file_unbuffered))); while (TRY(file->can_read_line())) { auto line = TRY(file->read_line(buffer)); @@ -148,7 +149,7 @@ static ErrorOr mount_all() static ErrorOr print_mounts() { // Output info about currently mounted filesystems. - auto df = TRY(Core::Stream::File::open("/sys/kernel/df"sv, Core::Stream::OpenMode::Read)); + auto df = TRY(Core::File::open("/sys/kernel/df"sv, Core::File::OpenMode::Read)); auto content = TRY(df->read_until_eof()); auto json = TRY(JsonValue::from_string(content)); diff --git a/Userland/Utilities/netstat.cpp b/Userland/Utilities/netstat.cpp index 0f5038591e2110..6e99860a456b57 100644 --- a/Userland/Utilities/netstat.cpp +++ b/Userland/Utilities/netstat.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -154,7 +155,7 @@ ErrorOr serenity_main(Main::Arguments arguments) } if (!has_protocol_flag || flag_tcp) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/net/tcp"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/net/tcp"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json_or_error = JsonValue::from_string(file_contents); if (json_or_error.is_error()) { @@ -246,7 +247,7 @@ ErrorOr serenity_main(Main::Arguments arguments) } if (!has_protocol_flag || flag_udp) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/net/udp"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/net/udp"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); diff --git a/Userland/Utilities/nologin.cpp b/Userland/Utilities/nologin.cpp index 8c187d97644d52..c28572c6263527 100644 --- a/Userland/Utilities/nologin.cpp +++ b/Userland/Utilities/nologin.cpp @@ -5,6 +5,7 @@ */ #include +#include #include #include #include @@ -13,7 +14,7 @@ ErrorOr serenity_main(Main::Arguments) { TRY(Core::System::pledge("stdio rpath"sv)); - auto file_or_error = Core::Stream::File::open("/etc/nologin"sv, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open("/etc/nologin"sv, Core::File::OpenMode::Read); if (file_or_error.is_error()) { outln("This account is currently not available."); } else { diff --git a/Userland/Utilities/nproc.cpp b/Userland/Utilities/nproc.cpp index 54f730fe7ad6f1..4d8eccc00b7575 100644 --- a/Userland/Utilities/nproc.cpp +++ b/Userland/Utilities/nproc.cpp @@ -5,6 +5,7 @@ */ #include +#include #include #include #include @@ -12,7 +13,7 @@ ErrorOr serenity_main(Main::Arguments) { TRY(Core::System::pledge("stdio rpath")); - auto file = TRY(Core::Stream::File::open("/sys/kernel/cpuinfo"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/cpuinfo"sv, Core::File::OpenMode::Read)); auto buffer = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(buffer)); diff --git a/Userland/Utilities/pmap.cpp b/Userland/Utilities/pmap.cpp index 7a95a43231a290..49bb7d8153357a 100644 --- a/Userland/Utilities/pmap.cpp +++ b/Userland/Utilities/pmap.cpp @@ -8,6 +8,7 @@ #include #include #include +#include #include #include #include @@ -26,7 +27,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(pid, "PID", "PID", Core::ArgsParser::Required::Yes); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open(DeprecatedString::formatted("/proc/{}/vm", pid), Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(DeprecatedString::formatted("/proc/{}/vm", pid), Core::File::OpenMode::Read)); outln("{}:", pid); diff --git a/Userland/Utilities/pro.cpp b/Userland/Utilities/pro.cpp index 0f8b8dda3bece7..150f80394dd946 100644 --- a/Userland/Utilities/pro.cpp +++ b/Userland/Utilities/pro.cpp @@ -242,7 +242,7 @@ ErrorOr serenity_main(Main::Arguments arguments) RefPtr request; auto protocol_client = TRY(Protocol::RequestClient::try_create()); - auto output_stream = ConditionalOutputStream { [&] { return should_save_stream_data; }, TRY(Core::Stream::File::adopt_fd(output_fd, Core::Stream::OpenMode::Write)) }; + auto output_stream = ConditionalOutputStream { [&] { return should_save_stream_data; }, TRY(Core::File::adopt_fd(output_fd, Core::File::OpenMode::Write)) }; // https://httpwg.org/specs/rfc9110.html#authentication auto const has_credentials = !credentials.is_empty(); diff --git a/Userland/Utilities/reboot.cpp b/Userland/Utilities/reboot.cpp index 2c24c9b22559b4..2aea62645b7f22 100644 --- a/Userland/Utilities/reboot.cpp +++ b/Userland/Utilities/reboot.cpp @@ -5,12 +5,13 @@ * SPDX-License-Identifier: BSD-2-Clause */ +#include #include #include ErrorOr serenity_main(Main::Arguments) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/power_state"sv, Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open("/sys/kernel/power_state"sv, Core::File::OpenMode::Write)); const DeprecatedString file_contents = "1"; TRY(file->write(file_contents.bytes())); diff --git a/Userland/Utilities/route.cpp b/Userland/Utilities/route.cpp index 93c89a0167a273..84fe136dcd3987 100644 --- a/Userland/Utilities/route.cpp +++ b/Userland/Utilities/route.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -89,7 +90,7 @@ ErrorOr serenity_main(Main::Arguments arguments) }; if (modify_action.is_empty()) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/net/route"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/net/route"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); diff --git a/Userland/Utilities/sed.cpp b/Userland/Utilities/sed.cpp index d25dd969a7fada..cd1d4363e2770b 100644 --- a/Userland/Utilities/sed.cpp +++ b/Userland/Utilities/sed.cpp @@ -8,6 +8,7 @@ #include #include #include +#include #include #include #include @@ -116,17 +117,17 @@ ErrorOr serenity_main(Main::Arguments args) auto command = TRY(parse_command(command_input)); - Optional> maybe_output_file; + Optional> maybe_output_file; if (command.output_filepath.has_value()) - maybe_output_file = TRY(Core::Stream::File::open_file_or_standard_stream(command.output_filepath.release_value(), Core::Stream::OpenMode::Write)); + maybe_output_file = TRY(Core::File::open_file_or_standard_stream(command.output_filepath.release_value(), Core::File::OpenMode::Write)); if (filepaths.is_empty()) filepaths = { "-"sv }; Array buffer {}; for (auto const& filepath : filepaths) { - auto file_unbuffered = TRY(Core::Stream::File::open_file_or_standard_stream(filepath, Core::Stream::OpenMode::Read)); - auto file = TRY(Core::Stream::BufferedFile::create(move(file_unbuffered))); + auto file_unbuffered = TRY(Core::File::open_file_or_standard_stream(filepath, Core::File::OpenMode::Read)); + auto file = TRY(Core::BufferedFile::create(move(file_unbuffered))); while (!file->is_eof()) { auto line = TRY(file->read_line(buffer)); diff --git a/Userland/Utilities/shot.cpp b/Userland/Utilities/shot.cpp index 3ecbfff3cb3db4..f39f8df3624c60 100644 --- a/Userland/Utilities/shot.cpp +++ b/Userland/Utilities/shot.cpp @@ -161,7 +161,7 @@ ErrorOr serenity_main(Main::Arguments arguments) if (edit_image) output_path = Core::DateTime::now().to_deprecated_string("/tmp/screenshot-%Y-%m-%d-%H-%M-%S.png"sv); - auto file_or_error = Core::Stream::File::open(output_path, Core::Stream::OpenMode::ReadWrite); + auto file_or_error = Core::File::open(output_path, Core::File::OpenMode::ReadWrite); if (file_or_error.is_error()) { warnln("Could not open '{}' for writing: {}", output_path, file_or_error.error()); return 1; diff --git a/Userland/Utilities/shuf.cpp b/Userland/Utilities/shuf.cpp index 30a1c9f379b214..1b0c94bf992ade 100644 --- a/Userland/Utilities/shuf.cpp +++ b/Userland/Utilities/shuf.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -28,7 +29,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read)); ByteBuffer buffer = TRY(file->read_until_eof()); u8 input_delimiter = is_zero_terminated ? '\0' : '\n'; diff --git a/Userland/Utilities/shutdown.cpp b/Userland/Utilities/shutdown.cpp index 5555ba369361ca..968e88e8356aa8 100644 --- a/Userland/Utilities/shutdown.cpp +++ b/Userland/Utilities/shutdown.cpp @@ -6,6 +6,7 @@ * SPDX-License-Identifier: BSD-2-Clause */ +#include #include #include #include @@ -14,7 +15,7 @@ ErrorOr serenity_main(Main::Arguments) { - auto file = TRY(Core::Stream::File::open("/sys/kernel/power_state"sv, Core::Stream::OpenMode::Write)); + auto file = TRY(Core::File::open("/sys/kernel/power_state"sv, Core::File::OpenMode::Write)); const DeprecatedString file_contents = "2"; TRY(file->write(file_contents.bytes())); diff --git a/Userland/Utilities/sort.cpp b/Userland/Utilities/sort.cpp index 46d0a9ca805bf3..d91cbf3d08f53c 100644 --- a/Userland/Utilities/sort.cpp +++ b/Userland/Utilities/sort.cpp @@ -9,6 +9,7 @@ #include #include #include +#include #include #include #include @@ -60,8 +61,8 @@ struct Options { static ErrorOr load_file(Options options, StringView filename, Vector& lines, HashTable& seen) { - auto file = TRY(Core::Stream::BufferedFile::create( - TRY(Core::Stream::File::open_file_or_standard_stream(filename, Core::Stream::OpenMode::Read)))); + auto file = TRY(Core::BufferedFile::create( + TRY(Core::File::open_file_or_standard_stream(filename, Core::File::OpenMode::Read)))); // FIXME: Unlimited line length auto buffer = TRY(ByteBuffer::create_uninitialized(4096)); diff --git a/Userland/Utilities/sql.cpp b/Userland/Utilities/sql.cpp index 9158cc793e9785..31be2c664f09e0 100644 --- a/Userland/Utilities/sql.cpp +++ b/Userland/Utilities/sql.cpp @@ -152,7 +152,7 @@ class SQLRepl { NonnullRefPtr m_sql_client; SQL::ConnectionID m_connection_id { 0 }; Core::EventLoop& m_loop; - OwnPtr m_input_file { nullptr }; + OwnPtr m_input_file { nullptr }; bool m_quit_when_files_read { false }; Vector m_input_file_chain {}; Array m_buffer {}; @@ -161,13 +161,13 @@ class SQLRepl { { if (!m_input_file && !m_input_file_chain.is_empty()) { auto file_name = m_input_file_chain.take_first(); - auto file_or_error = Core::Stream::File::open(file_name, Core::Stream::OpenMode::Read); + auto file_or_error = Core::File::open(file_name, Core::File::OpenMode::Read); if (file_or_error.is_error()) { warnln("Input file {} could not be opened: {}", file_name, file_or_error.error()); return {}; } - auto buffered_file_or_error = Core::Stream::BufferedFile::create(file_or_error.release_value()); + auto buffered_file_or_error = Core::BufferedFile::create(file_or_error.release_value()); if (buffered_file_or_error.is_error()) { warnln("Input file {} could not be buffered: {}", file_name, buffered_file_or_error.error()); return {}; diff --git a/Userland/Utilities/strace.cpp b/Userland/Utilities/strace.cpp index 1996e5d12d0438..a20ae8a55a5b1a 100644 --- a/Userland/Utilities/strace.cpp +++ b/Userland/Utilities/strace.cpp @@ -11,6 +11,7 @@ #include #include #include +#include #include #include #include @@ -836,8 +837,8 @@ ErrorOr serenity_main(Main::Arguments arguments) parser.parse(arguments); auto trace_file = output_filename.is_empty() - ? TRY(Core::Stream::File::standard_error()) - : TRY(Core::Stream::File::open(output_filename, Core::Stream::OpenMode::Write)); + ? TRY(Core::File::standard_error()) + : TRY(Core::File::open(output_filename, Core::File::OpenMode::Write)); auto parse_syscalls = [](char const* option, auto& hash_table) { if (option != nullptr) { diff --git a/Userland/Utilities/strings.cpp b/Userland/Utilities/strings.cpp index 531db3183806c8..2fb364c97f3bdc 100644 --- a/Userland/Utilities/strings.cpp +++ b/Userland/Utilities/strings.cpp @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -67,7 +68,7 @@ static ErrorOr process_strings_in_file(StringView path, bool show_paths, S { Array buffer; Vector output_characters; - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read)); size_t processed_characters = 0; size_t string_offset_position = 0; bool did_show_path = false; diff --git a/Userland/Utilities/sysctl.cpp b/Userland/Utilities/sysctl.cpp index c234edbf857993..3834f9aa125dd2 100644 --- a/Userland/Utilities/sysctl.cpp +++ b/Userland/Utilities/sysctl.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include @@ -15,7 +16,7 @@ static bool s_set_variable = false; static DeprecatedString get_variable(StringView name) { auto path = DeprecatedString::formatted("/sys/kernel/variables/{}", name); - auto file = Core::Stream::File::open(path, Core::Stream::OpenMode::Read); + auto file = Core::File::open(path, Core::File::OpenMode::Read); if (file.is_error()) { warnln("Failed to open {}: {}", path, file.error()); return {}; @@ -43,7 +44,7 @@ static bool write_variable(StringView name, StringView value) if (old_value.is_null()) return false; auto path = DeprecatedString::formatted("/sys/kernel/variables/{}", name); - auto file = Core::Stream::File::open(path, Core::Stream::OpenMode::Write); + auto file = Core::File::open(path, Core::File::OpenMode::Write); if (file.is_error()) { warnln("Failed to open {}: {}", path, file.error()); return false; diff --git a/Userland/Utilities/tail.cpp b/Userland/Utilities/tail.cpp index 96b5fe0234d4fb..658c4aba34b785 100644 --- a/Userland/Utilities/tail.cpp +++ b/Userland/Utilities/tail.cpp @@ -6,13 +6,14 @@ #include #include +#include #include #include #include #define DEFAULT_LINE_COUNT 10 -static ErrorOr tail_from_pos(Core::Stream::File& file, off_t startline) +static ErrorOr tail_from_pos(Core::File& file, off_t startline) { TRY(file.seek(startline + 1, SeekMode::SetPosition)); auto buffer = TRY(file.read_until_eof()); @@ -20,7 +21,7 @@ static ErrorOr tail_from_pos(Core::Stream::File& file, off_t startline) return {}; } -static ErrorOr find_seek_pos(Core::Stream::File& file, int wanted_lines) +static ErrorOr find_seek_pos(Core::File& file, int wanted_lines) { // Rather than reading the whole file, start at the end and work backwards, // stopping when we've found the number of lines we want. @@ -61,7 +62,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_positional_argument(file, "File path", "file", Core::ArgsParser::Required::No); args_parser.parse(arguments); - auto f = TRY(Core::Stream::File::open_file_or_standard_stream(file, Core::Stream::OpenMode::Read)); + auto f = TRY(Core::File::open_file_or_standard_stream(file, Core::File::OpenMode::Read)); if (!follow) TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/tar.cpp b/Userland/Utilities/tar.cpp index c7af3500051eb6..2a156b9d025ace 100644 --- a/Userland/Utilities/tar.cpp +++ b/Userland/Utilities/tar.cpp @@ -64,7 +64,7 @@ ErrorOr serenity_main(Main::Arguments arguments) if (!directory.is_empty()) TRY(Core::System::chdir(directory)); - NonnullOwnPtr input_stream = TRY(Core::Stream::File::open_file_or_standard_stream(archive_file, Core::Stream::OpenMode::Read)); + NonnullOwnPtr input_stream = TRY(Core::File::open_file_or_standard_stream(archive_file, Core::File::OpenMode::Read)); if (gzip) input_stream = make(move(input_stream)); @@ -206,10 +206,10 @@ ErrorOr serenity_main(Main::Arguments arguments) return 1; } - NonnullOwnPtr output_stream = TRY(Core::Stream::File::standard_output()); + NonnullOwnPtr output_stream = TRY(Core::File::standard_output()); if (!archive_file.is_empty()) - output_stream = TRY(Core::Stream::File::open(archive_file, Core::Stream::OpenMode::Write)); + output_stream = TRY(Core::File::open(archive_file, Core::File::OpenMode::Write)); if (!directory.is_empty()) TRY(Core::System::chdir(directory)); diff --git a/Userland/Utilities/test-fuzz.cpp b/Userland/Utilities/test-fuzz.cpp index a7539d17745b62..8f01380dd03858 100644 --- a/Userland/Utilities/test-fuzz.cpp +++ b/Userland/Utilities/test-fuzz.cpp @@ -6,6 +6,7 @@ #include #include +#include #include #include #include @@ -156,7 +157,7 @@ ErrorOr serenity_main(Main::Arguments arguments) auto fn = parse_target_name(type); - auto file = TRY(Core::Stream::File::open(filename, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(filename, Core::File::OpenMode::Read)); auto input = TRY(file->read_until_eof()); return fn(input.data(), input.size()); diff --git a/Userland/Utilities/tsort.cpp b/Userland/Utilities/tsort.cpp index 201e063698527b..f51b2f96f6fcaa 100644 --- a/Userland/Utilities/tsort.cpp +++ b/Userland/Utilities/tsort.cpp @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -103,7 +104,7 @@ ErrorOr serenity_main(Main::Arguments arguments) args_parser.add_option(quiet, "Suppress warnings about cycles", "quiet", 'q'); args_parser.parse(arguments); - auto file = TRY(Core::Stream::File::open_file_or_standard_stream(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open_file_or_standard_stream(path, Core::File::OpenMode::Read)); auto input_bytes = TRY(file->read_until_eof()); auto inputs = StringView(input_bytes).split_view_if(is_ascii_space); diff --git a/Userland/Utilities/uniq.cpp b/Userland/Utilities/uniq.cpp index c8166e49681392..35b502e4c83dd4 100644 --- a/Userland/Utilities/uniq.cpp +++ b/Userland/Utilities/uniq.cpp @@ -8,11 +8,12 @@ #include #include #include +#include #include #include #include -static ErrorOr write_line_content(StringView line, size_t count, bool duplicates_only, bool print_count, Core::Stream::File& outfile) +static ErrorOr write_line_content(StringView line, size_t count, bool duplicates_only, bool print_count, Core::File& outfile) { if (duplicates_only && count <= 1) return {}; @@ -79,8 +80,8 @@ ErrorOr serenity_main(Main::Arguments arguments) return 0; } - auto infile = TRY(Core::Stream::BufferedFile::create(TRY(Core::Stream::File::open_file_or_standard_stream(inpath, Core::Stream::OpenMode::Read)))); - auto outfile = TRY(Core::Stream::File::open_file_or_standard_stream(outpath, Core::Stream::OpenMode::Write)); + auto infile = TRY(Core::BufferedFile::create(TRY(Core::File::open_file_or_standard_stream(inpath, Core::File::OpenMode::Read)))); + auto outfile = TRY(Core::File::open_file_or_standard_stream(outpath, Core::File::OpenMode::Write)); size_t count = 0; ByteBuffer previous_buf = TRY(ByteBuffer::create_uninitialized(1024)); diff --git a/Userland/Utilities/uptime.cpp b/Userland/Utilities/uptime.cpp index 265fda6be53bfd..f70cdaf8c45f23 100644 --- a/Userland/Utilities/uptime.cpp +++ b/Userland/Utilities/uptime.cpp @@ -6,6 +6,7 @@ */ #include +#include #include #include #include @@ -14,7 +15,7 @@ ErrorOr serenity_main(Main::Arguments) { TRY(Core::System::pledge("stdio rpath")); - auto file = TRY(Core::Stream::File::open("/sys/kernel/uptime"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/sys/kernel/uptime"sv, Core::File::OpenMode::Read)); TRY(Core::System::pledge("stdio")); diff --git a/Userland/Utilities/utmpupdate.cpp b/Userland/Utilities/utmpupdate.cpp index cc7e282488f3d7..0bca719706884c 100644 --- a/Userland/Utilities/utmpupdate.cpp +++ b/Userland/Utilities/utmpupdate.cpp @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -42,7 +43,7 @@ ErrorOr serenity_main(Main::Arguments arguments) dbgln("Updating utmp from UID={} GID={} EGID={} PID={}", getuid(), getgid(), getegid(), pid); - auto file = TRY(Core::Stream::File::open("/var/run/utmp"sv, Core::Stream::OpenMode::ReadWrite)); + auto file = TRY(Core::File::open("/var/run/utmp"sv, Core::File::OpenMode::ReadWrite)); auto file_contents = TRY(file->read_until_eof()); auto previous_json = TRY(JsonValue::from_string(file_contents)); diff --git a/Userland/Utilities/w.cpp b/Userland/Utilities/w.cpp index 96f3854b2d946e..e92f791055dc22 100644 --- a/Userland/Utilities/w.cpp +++ b/Userland/Utilities/w.cpp @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -25,7 +26,7 @@ ErrorOr serenity_main(Main::Arguments) TRY(Core::System::unveil("/sys/kernel/processes", "r")); TRY(Core::System::unveil(nullptr, nullptr)); - auto file = TRY(Core::Stream::File::open("/var/run/utmp"sv, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open("/var/run/utmp"sv, Core::File::OpenMode::Read)); auto file_contents = TRY(file->read_until_eof()); auto json = TRY(JsonValue::from_string(file_contents)); if (!json.is_object()) { diff --git a/Userland/Utilities/wasm.cpp b/Userland/Utilities/wasm.cpp index 5dababeb101278..d82ff82fa2db21 100644 --- a/Userland/Utilities/wasm.cpp +++ b/Userland/Utilities/wasm.cpp @@ -7,6 +7,7 @@ #include #include +#include #include #include #include @@ -338,7 +339,7 @@ ErrorOr serenity_main(Main::Arguments arguments) if (!parse_result.has_value()) return 1; - g_stdout = TRY(Core::Stream::File::standard_output()); + g_stdout = TRY(Core::File::standard_output()); g_printer = TRY(try_make(*g_stdout)); if (print && !attempt_instantiate) { diff --git a/Userland/Utilities/xml.cpp b/Userland/Utilities/xml.cpp index 7be705dd6aed80..78796142c521ca 100644 --- a/Userland/Utilities/xml.cpp +++ b/Userland/Utilities/xml.cpp @@ -10,6 +10,7 @@ #include #include #include +#include #include #include #include @@ -371,7 +372,7 @@ static auto parse(StringView contents) if (url.scheme() != "file") return Error::from_string_literal("NYI: Nonlocal entity"); - auto file = TRY(Core::Stream::File::open(url.path(), Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(url.path(), Core::File::OpenMode::Read)); return DeprecatedString::copy(TRY(file->read_until_eof())); }, }, @@ -440,7 +441,7 @@ static void do_run_tests(XML::Document& document) continue; } - auto file_result = Core::Stream::File::open(url.path(), Core::Stream::OpenMode::Read); + auto file_result = Core::File::open(url.path(), Core::File::OpenMode::Read); if (file_result.is_error()) { warnln("Read error for {}: {}", url.path(), file_result.error()); s_test_results.set(url.path(), TestResult::RunnerFailed); @@ -468,7 +469,7 @@ static void do_run_tests(XML::Document& document) auto out = suite.attributes.find("OUTPUT"); if (out != suite.attributes.end()) { auto out_path = LexicalPath::join(test_base_path, out->value).string(); - auto file_result = Core::Stream::File::open(out_path, Core::Stream::OpenMode::Read); + auto file_result = Core::File::open(out_path, Core::File::OpenMode::Read); if (file_result.is_error()) { warnln("Read error for {}: {}", out_path, file_result.error()); s_test_results.set(url.path(), TestResult::RunnerFailed); @@ -516,7 +517,7 @@ ErrorOr serenity_main(Main::Arguments arguments) parser.parse(arguments); s_path = Core::DeprecatedFile::real_path_for(filename); - auto file = TRY(Core::Stream::File::open(s_path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(s_path, Core::File::OpenMode::Read)); auto contents = TRY(file->read_until_eof()); auto xml_parser = parse(contents); diff --git a/Userland/Utilities/zip.cpp b/Userland/Utilities/zip.cpp index 7a9b88deff88c6..12679b67e0cafd 100644 --- a/Userland/Utilities/zip.cpp +++ b/Userland/Utilities/zip.cpp @@ -12,6 +12,7 @@ #include #include #include +#include #include #include #include @@ -50,12 +51,12 @@ ErrorOr serenity_main(Main::Arguments arguments) } outln("Archive: {}", zip_file_path); - auto file_stream = TRY(Core::Stream::File::open(zip_file_path, Core::Stream::OpenMode::Write)); + auto file_stream = TRY(Core::File::open(zip_file_path, Core::File::OpenMode::Write)); Archive::ZipOutputStream zip_stream(move(file_stream)); auto add_file = [&](DeprecatedString path) -> ErrorOr { auto canonicalized_path = LexicalPath::canonicalized_path(path); - auto file = TRY(Core::Stream::File::open(path, Core::Stream::OpenMode::Read)); + auto file = TRY(Core::File::open(path, Core::File::OpenMode::Read)); auto file_buffer = TRY(file->read_until_eof()); Archive::ZipMember member {}; member.name = TRY(String::from_deprecated_string(canonicalized_path));