diff options
author | sin-ack <sin-ack@users.noreply.github.com> | 2021-04-25 08:04:17 +0200 |
---|---|---|
committer | Andreas Kling <kling@serenityos.org> | 2021-04-25 09:29:49 +0200 |
commit | 59218007a3034c7770b7fa5277ce1eae85bb1c07 (patch) | |
tree | 58595851a1de3e3921f453a1500e8991c9f1e31a | |
parent | f89c60664c001e72128b0142d972f31cb13590d3 (diff) | |
download | serenity-59218007a3034c7770b7fa5277ce1eae85bb1c07.zip |
IPCCompiler: Use string hashes for IPC endpoint magic
This patch removes the IPC endpoint numbers that needed to be specified
in the IPC files. Since the string hash is a (hopefully) collision free
number that depends on the name of the endpoint, we now use that
instead. :^)
25 files changed, 25 insertions, 29 deletions
diff --git a/Userland/DevTools/HackStudio/LanguageServers/LanguageClient.ipc b/Userland/DevTools/HackStudio/LanguageServers/LanguageClient.ipc index 889a37623e..8a20b31fb8 100644 --- a/Userland/DevTools/HackStudio/LanguageServers/LanguageClient.ipc +++ b/Userland/DevTools/HackStudio/LanguageServers/LanguageClient.ipc @@ -1,4 +1,4 @@ -endpoint LanguageClient = 8002 +endpoint LanguageClient { AutoCompleteSuggestions(Vector<GUI::AutocompleteProvider::Entry> suggestions) =| DeclarationLocation(GUI::AutocompleteProvider::ProjectLocation location) =| diff --git a/Userland/DevTools/HackStudio/LanguageServers/LanguageServer.ipc b/Userland/DevTools/HackStudio/LanguageServers/LanguageServer.ipc index c8d804410a..938d616be2 100644 --- a/Userland/DevTools/HackStudio/LanguageServers/LanguageServer.ipc +++ b/Userland/DevTools/HackStudio/LanguageServers/LanguageServer.ipc @@ -1,4 +1,4 @@ -endpoint LanguageServer = 8001 +endpoint LanguageServer { Greet(String project_root) => () diff --git a/Userland/DevTools/IPCCompiler/main.cpp b/Userland/DevTools/IPCCompiler/main.cpp index 1f5aa08a25..fbd3012de1 100644 --- a/Userland/DevTools/IPCCompiler/main.cpp +++ b/Userland/DevTools/IPCCompiler/main.cpp @@ -164,11 +164,7 @@ int main(int argc, char** argv) lexer.consume_specific("endpoint"); consume_whitespace(); endpoints.last().name = lexer.consume_while([](char ch) { return !isspace(ch); }); - consume_whitespace(); - assert_specific('='); - consume_whitespace(); - auto magic_string = lexer.consume_while([](char ch) { return !isspace(ch) && ch != '{'; }); - endpoints.last().magic = magic_string.to_int().value(); + endpoints.last().magic = Traits<String>::hash(endpoints.last().name); consume_whitespace(); assert_specific('{'); parse_messages(); diff --git a/Userland/Services/AudioServer/AudioClient.ipc b/Userland/Services/AudioServer/AudioClient.ipc index b43db52dd0..d444c6c007 100644 --- a/Userland/Services/AudioServer/AudioClient.ipc +++ b/Userland/Services/AudioServer/AudioClient.ipc @@ -1,4 +1,4 @@ -endpoint AudioClient = 82 +endpoint AudioClient { FinishedPlayingBuffer(i32 buffer_id) =| MutedStateChanged(bool muted) =| diff --git a/Userland/Services/AudioServer/AudioServer.ipc b/Userland/Services/AudioServer/AudioServer.ipc index a52a837f41..f268e747bd 100644 --- a/Userland/Services/AudioServer/AudioServer.ipc +++ b/Userland/Services/AudioServer/AudioServer.ipc @@ -1,4 +1,4 @@ -endpoint AudioServer = 85 +endpoint AudioServer { // Basic protocol Greet() => () diff --git a/Userland/Services/Clipboard/ClipboardClient.ipc b/Userland/Services/Clipboard/ClipboardClient.ipc index 50d695e92a..d0d3a061a6 100644 --- a/Userland/Services/Clipboard/ClipboardClient.ipc +++ b/Userland/Services/Clipboard/ClipboardClient.ipc @@ -1,4 +1,4 @@ -endpoint ClipboardClient = 804 +endpoint ClipboardClient { ClipboardDataChanged([UTF8] String mime_type) =| } diff --git a/Userland/Services/Clipboard/ClipboardServer.ipc b/Userland/Services/Clipboard/ClipboardServer.ipc index 1c6372cb4e..9fe63e2fc0 100644 --- a/Userland/Services/Clipboard/ClipboardServer.ipc +++ b/Userland/Services/Clipboard/ClipboardServer.ipc @@ -1,4 +1,4 @@ -endpoint ClipboardServer = 802 +endpoint ClipboardServer { Greet() => () diff --git a/Userland/Services/ImageDecoder/ImageDecoderClient.ipc b/Userland/Services/ImageDecoder/ImageDecoderClient.ipc index 49dc3862c2..4b486e38b2 100644 --- a/Userland/Services/ImageDecoder/ImageDecoderClient.ipc +++ b/Userland/Services/ImageDecoder/ImageDecoderClient.ipc @@ -1,4 +1,4 @@ -endpoint ImageDecoderClient = 7002 +endpoint ImageDecoderClient { Dummy() =| } diff --git a/Userland/Services/ImageDecoder/ImageDecoderServer.ipc b/Userland/Services/ImageDecoder/ImageDecoderServer.ipc index c2808822df..41f135c4b0 100644 --- a/Userland/Services/ImageDecoder/ImageDecoderServer.ipc +++ b/Userland/Services/ImageDecoder/ImageDecoderServer.ipc @@ -1,4 +1,4 @@ -endpoint ImageDecoderServer = 7001 +endpoint ImageDecoderServer { Greet() => () diff --git a/Userland/Services/LaunchServer/LaunchClient.ipc b/Userland/Services/LaunchServer/LaunchClient.ipc index abd4e9ba3d..2e4f324702 100644 --- a/Userland/Services/LaunchServer/LaunchClient.ipc +++ b/Userland/Services/LaunchServer/LaunchClient.ipc @@ -1,4 +1,4 @@ -endpoint LaunchClient = 102 +endpoint LaunchClient { Dummy() =| } diff --git a/Userland/Services/LaunchServer/LaunchServer.ipc b/Userland/Services/LaunchServer/LaunchServer.ipc index 34ac5a6db3..7d160f484f 100644 --- a/Userland/Services/LaunchServer/LaunchServer.ipc +++ b/Userland/Services/LaunchServer/LaunchServer.ipc @@ -1,4 +1,4 @@ -endpoint LaunchServer = 101 +endpoint LaunchServer { Greet() => () OpenURL(URL url, String handler_name) => (bool response) diff --git a/Userland/Services/LookupServer/LookupClient.ipc b/Userland/Services/LookupServer/LookupClient.ipc index 6455140d91..567b6fcb82 100644 --- a/Userland/Services/LookupServer/LookupClient.ipc +++ b/Userland/Services/LookupServer/LookupClient.ipc @@ -1,4 +1,4 @@ -endpoint LookupClient = 9002 +endpoint LookupClient { Dummy() =| } diff --git a/Userland/Services/LookupServer/LookupServer.ipc b/Userland/Services/LookupServer/LookupServer.ipc index e23182346a..499e2290a4 100644 --- a/Userland/Services/LookupServer/LookupServer.ipc +++ b/Userland/Services/LookupServer/LookupServer.ipc @@ -1,4 +1,4 @@ -endpoint LookupServer = 9001 +endpoint LookupServer { LookupName(String name) => (int code, Vector<String> addresses) LookupAddress(String address) => (int code, String name) diff --git a/Userland/Services/NotificationServer/NotificationClient.ipc b/Userland/Services/NotificationServer/NotificationClient.ipc index fc85168f76..9742ea513d 100644 --- a/Userland/Services/NotificationServer/NotificationClient.ipc +++ b/Userland/Services/NotificationServer/NotificationClient.ipc @@ -1,4 +1,4 @@ -endpoint NotificationClient = 92 +endpoint NotificationClient { Dummy() =| } diff --git a/Userland/Services/NotificationServer/NotificationServer.ipc b/Userland/Services/NotificationServer/NotificationServer.ipc index 830e9d6312..2a3c61932b 100644 --- a/Userland/Services/NotificationServer/NotificationServer.ipc +++ b/Userland/Services/NotificationServer/NotificationServer.ipc @@ -1,4 +1,4 @@ -endpoint NotificationServer = 95 +endpoint NotificationServer { // Basic protocol Greet() => () diff --git a/Userland/Services/ProtocolServer/ProtocolClient.ipc b/Userland/Services/ProtocolServer/ProtocolClient.ipc index 88f4cfc96d..37707472a6 100644 --- a/Userland/Services/ProtocolServer/ProtocolClient.ipc +++ b/Userland/Services/ProtocolServer/ProtocolClient.ipc @@ -1,4 +1,4 @@ -endpoint ProtocolClient = 13 +endpoint ProtocolClient { // Download notifications DownloadProgress(i32 download_id, Optional<u32> total_size, u32 downloaded_size) =| diff --git a/Userland/Services/ProtocolServer/ProtocolServer.ipc b/Userland/Services/ProtocolServer/ProtocolServer.ipc index 3a8ecdabad..2348e01af8 100644 --- a/Userland/Services/ProtocolServer/ProtocolServer.ipc +++ b/Userland/Services/ProtocolServer/ProtocolServer.ipc @@ -1,4 +1,4 @@ -endpoint ProtocolServer = 9 +endpoint ProtocolServer { // Basic protocol Greet() => () diff --git a/Userland/Services/SymbolServer/SymbolClient.ipc b/Userland/Services/SymbolServer/SymbolClient.ipc index 575a17f509..8b4a1be029 100644 --- a/Userland/Services/SymbolServer/SymbolClient.ipc +++ b/Userland/Services/SymbolServer/SymbolClient.ipc @@ -1,4 +1,4 @@ -endpoint SymbolClient = 4541511 +endpoint SymbolClient { Dummy() =| } diff --git a/Userland/Services/SymbolServer/SymbolServer.ipc b/Userland/Services/SymbolServer/SymbolServer.ipc index a578b01809..d1eed6ac03 100644 --- a/Userland/Services/SymbolServer/SymbolServer.ipc +++ b/Userland/Services/SymbolServer/SymbolServer.ipc @@ -1,4 +1,4 @@ -endpoint SymbolServer = 4541510 +endpoint SymbolServer { Greet() => () diff --git a/Userland/Services/WebContent/WebContentClient.ipc b/Userland/Services/WebContent/WebContentClient.ipc index c74abbc3af..c7aad24ca0 100644 --- a/Userland/Services/WebContent/WebContentClient.ipc +++ b/Userland/Services/WebContent/WebContentClient.ipc @@ -1,4 +1,4 @@ -endpoint WebContentClient = 90 +endpoint WebContentClient { DidStartLoading(URL url) =| DidFinishLoading(URL url) =| diff --git a/Userland/Services/WebContent/WebContentServer.ipc b/Userland/Services/WebContent/WebContentServer.ipc index 1b429c5579..1f6e30ce22 100644 --- a/Userland/Services/WebContent/WebContentServer.ipc +++ b/Userland/Services/WebContent/WebContentServer.ipc @@ -1,4 +1,4 @@ -endpoint WebContentServer = 89 +endpoint WebContentServer { Greet() => () diff --git a/Userland/Services/WindowServer/WindowClient.ipc b/Userland/Services/WindowServer/WindowClient.ipc index c7ffb91a5b..ea8f49635d 100644 --- a/Userland/Services/WindowServer/WindowClient.ipc +++ b/Userland/Services/WindowServer/WindowClient.ipc @@ -1,4 +1,4 @@ -endpoint WindowClient = 4 +endpoint WindowClient { Paint(i32 window_id, Gfx::IntSize window_size, Vector<Gfx::IntRect> rects) =| MouseMove(i32 window_id, Gfx::IntPoint mouse_position, u32 button, u32 buttons, u32 modifiers, i32 wheel_delta, bool is_drag, Vector<String> mime_types) =| diff --git a/Userland/Services/WindowServer/WindowManagerClient.ipc b/Userland/Services/WindowServer/WindowManagerClient.ipc index 4c3c9c071d..facda30d8e 100644 --- a/Userland/Services/WindowServer/WindowManagerClient.ipc +++ b/Userland/Services/WindowServer/WindowManagerClient.ipc @@ -1,4 +1,4 @@ -endpoint WindowManagerClient = 1872 +endpoint WindowManagerClient { WindowRemoved(i32 wm_id, i32 client_id, i32 window_id) =| WindowStateChanged(i32 wm_id, i32 client_id, i32 window_id, i32 parent_client_id, i32 parent_window_id, bool is_active, bool is_minimized, bool is_modal, bool is_frameless, i32 window_type, [UTF8] String title, Gfx::IntRect rect, i32 progress) =| diff --git a/Userland/Services/WindowServer/WindowManagerServer.ipc b/Userland/Services/WindowServer/WindowManagerServer.ipc index 2973d31241..e0c12f72c9 100644 --- a/Userland/Services/WindowServer/WindowManagerServer.ipc +++ b/Userland/Services/WindowServer/WindowManagerServer.ipc @@ -1,4 +1,4 @@ -endpoint WindowManagerServer = 1871 +endpoint WindowManagerServer { SetEventMask(u32 event_mask) => () SetManagerWindow(i32 window_id) => () diff --git a/Userland/Services/WindowServer/WindowServer.ipc b/Userland/Services/WindowServer/WindowServer.ipc index a0f5135d72..948e6c7bce 100644 --- a/Userland/Services/WindowServer/WindowServer.ipc +++ b/Userland/Services/WindowServer/WindowServer.ipc @@ -1,4 +1,4 @@ -endpoint WindowServer = 2 +endpoint WindowServer { Greet() => (Gfx::IntRect screen_rect, Core::AnonymousBuffer theme_buffer) |