Add .elf file to .gitignore

"function" is a bash builtin

.gitignore

@@ -1,3 +1,4 @@
 *.o
 *.exe
 *.res
+*.elf

README.md

@@ -62,7 +62,7 @@ The steps for MacOS are almost the same, but due to the way `$TMPDIR` works, you
 
 The script will add a LaunchAgent to your user, that will symlink the `$TMPDIR` directory to `/tmp/rpc-bridge/tmpdir`.
 
-*Note: You will need to launch the `bridge.exe` file manually in Wine atleast one time for it to register and launch automatically the next times.*
+*Note: You will need to launch the `bridge.exe` file manually in Wine at least once for it to register and launch automatically the next time.*
 
 ## Compiling from source
 

bridge.c

@@ -250,32 +250,36 @@ char *native_getenv(const char *name)
 void ConnectToSocket(int fd)
 {
 	print("Connecting to socket\n");
-    const char *runtime;
-    if (IsLinux)
-        runtime = native_getenv("XDG_RUNTIME_DIR");
-    else
-        runtime = native_getenv("TMPDIR");
-    if (runtime == NULL)
-    {
-        runtime = "/tmp/rpc-bridge/tmpdir";
-        print("IPC directory not set, fallback to /tmp/rpc-bridge/tmpdir\n");
+	const char *runtime;
+	if (IsLinux)
+		runtime = native_getenv("XDG_RUNTIME_DIR");
+	else
+	{
+		runtime = native_getenv("TMPDIR");
+		if (runtime == NULL)
+		{
+			runtime = "/tmp/rpc-bridge/tmpdir";
+			print("IPC directory not set, fallback to /tmp/rpc-bridge/tmpdir\n");
 
-        // Check if the directory exists
-        DWORD dwAttrib = GetFileAttributes(runtime);
-        if (dwAttrib == INVALID_FILE_ATTRIBUTES || !(dwAttrib & FILE_ATTRIBUTE_DIRECTORY))
-        {
-            print("IPC directory does not exist: %s. If you're on MacOS, see the github guide on how to install the launchd service.\n", runtime);
-            // Handle the case where the directory doesn't exist
-            // For example, create the directory
-            if (!RunningAsService)
-                MessageBox(NULL, "IPC directory does not exist",
-                           "Directory not found",
-                           MB_OK | MB_ICONSTOP);
-            ExitProcess(1);
-        }
-    }
+			// Check if the directory exists
+			DWORD dwAttrib = GetFileAttributes(runtime);
+			if (dwAttrib == INVALID_FILE_ATTRIBUTES || !(dwAttrib & FILE_ATTRIBUTE_DIRECTORY))
+			{
+				print("IPC directory does not exist: %s. If you're on MacOS, see the github guide on how to install the launchd service.\n", runtime);
+				// Handle the case where the directory doesn't exist
+				// For example, create the directory
 
-    print("IPC directory: %s\n", runtime);
+				int result = MessageBox(NULL, "IPC directory does not exist\nDo you want to open the installation guide?",
+										"Directory not found",
+										MB_YESNO | MB_ICONSTOP);
+				if (result == IDYES)
+					ShellExecute(NULL, "open", "https://enderice2.github.io/rpc-bridge/installation.html#macos", NULL, NULL, SW_SHOWNORMAL);
+				ExitProcess(1);
+			}
+		}
+	}
+
+	print("IPC directory: %s\n", runtime);
 
 	/* TODO: check for multiple discord instances and create a pipe for each */
 	const char *discordUnixPipes[] = {
@@ -500,7 +504,7 @@ NewConnection:
 		CreateNamedPipe("\\\\.\\pipe\\discord-ipc-0",
 						PIPE_ACCESS_DUPLEX,
 						PIPE_TYPE_BYTE | PIPE_READMODE_BYTE | PIPE_WAIT,
-						1, 1024, 1024, 0, NULL);
+						PIPE_UNLIMITED_INSTANCES, BUFFER_LENGTH, BUFFER_LENGTH, 0, NULL);
 
 	if (hPipe == INVALID_HANDLE_VALUE)
 	{

bridge.rc

@@ -2,8 +2,7 @@
 #include <winuser.h>
 #include <winresrc.h>
 
-#define VER_VERSION             1,1,0,0
-#define VER_VERSION_STR         "1.1\0"
+#include "resource.h"
 
 VS_VERSION_INFO VERSIONINFO
 FILEVERSION     VER_VERSION
@@ -33,6 +32,22 @@ BEGIN
 	END
 END
 
+IDR_MAINMENU MENU
+BEGIN
+	POPUP "&View"
+	BEGIN
+		MENUITEM "&Log", IDM_VIEW_LOG
+	END
+	POPUP "&Help"
+	BEGIN
+		MENUITEM "&Documentation", IDM_HELP_DOCUMENTATION
+		MENUITEM "&License", IDM_HELP_LICENSE
+		MENUITEM "&About", IDM_HELP_ABOUT
+	END
+END
+
+IDR_LICENSE_TXT RCDATA "LICENSE"
+
 IDI_ICON_128 ICON "bridge.ico"
 
 CREATEPROCESS_MANIFEST_RESOURCE_ID RT_MANIFEST bridge.manifest

build/launchd.sh

@@ -1,6 +1,6 @@
-#!/bin/sh
+#!/bin/bash
 
-# This script is used to create a LaunchAgent on MacOS, to support the service functionnality.
+# This script is used to create a LaunchAgent on MacOS, to support the service functionality.
 # Usage: ./launchd.sh (install|remove)
 
 SYMLINK=/tmp/rpc-bridge/tmpdir
@@ -16,7 +16,7 @@ function install() {
     if [ ! -d "$LOCATION" ]; then
         mkdir -p "$LOCATION"
     fi
-    
+
     # Link script
     if [ -f "$SCRIPT" ]; then
         rm -f "$SCRIPT"
@@ -27,9 +27,9 @@ if [ ! -d "\$TARGET_DIR" ]; then
     mkdir -p "\$TARGET_DIR"
 fi
 rm -rf "\$TARGET_DIR"
-    ln -s "\$TMPDIR" "\$TARGET_DIR"" > "$SCRIPT"
+ln -s "\$TMPDIR" "\$TARGET_DIR"" > "$SCRIPT"
     chmod +x "$SCRIPT"
-    
+
     # LaunchAgent
     if [ -f "$AGENT" ]; then
         rm -f "$AGENT"
@@ -50,13 +50,12 @@ rm -rf "\$TARGET_DIR"
 </plist>" > "$AGENT"
     launchctl load "$AGENT"
     echo "LaunchAgent has been installed."
-    
 }
 
 function remove() {
-    rm -rf "$SYMLINK"
-    rm -rf "$LOCATION"
+    rm -f "$SYMLINK"
     rm -f "$SCRIPT"
+    rmdir "$LOCATION"
     if [ -f "$AGENT" ]; then
         launchctl unload "$AGENT"
     fi
@@ -81,4 +80,4 @@ case $1 in
         echo "Invalid argument. Please use 'install' or 'remove'."
         exit 1
     ;;
-esac
+esac

docs/installation.md

@@ -64,14 +64,16 @@ This method is recommended because it's easier to manage.
 
 ## macOS
 
-If using the default Wine prefix (`~/.wine`), you can follow the same steps as in Wine.
+The steps for MacOS are almost the same, but due to the way `$TMPDIR` works, you will have to install a **LaunchAgent**.
 
-Registering as a service [is not supported](https://github.com/EnderIce2/rpc-bridge/issues/1#issuecomment-2103423242 "Bridge can't get $TMPDIR unless is set with --rpc. See note below").
+- Download the latest build from the [releases](https://github.com/EnderIce2/rpc-bridge/releases)
+- Open the archive and make the `launchd.sh` script executable by doing: `chmod +x launchd.sh`
+- To **install** the LaunchAgent, run `./launchd install` and to **remove** it simply run `./launchd remove`.
+
+The script will add a LaunchAgent to your user, that will symlink the `$TMPDIR` directory to `/tmp/rpc-bridge/tmpdir`.
 
 !!! info "Note"
-
-    Since macOS doesn't have `/proc/self/environ` and `$TMPDIR` is neither always set nor the same after reboots, so you will need to [add `--rpc "$TMPDIR"` to the `bridge.exe` arguments](https://github.com/EnderIce2/rpc-bridge/issues/1#issuecomment-2104797235).  
-    If you don't encounter any issues, you can ignore this step.
+    You will need to launch the `bridge.exe` file manually in Wine at least once for it to register and launch automatically the next time.
 
 ## Run without installing the service
 

docs/usage.md

@@ -11,7 +11,10 @@
 ## Commands
 
 - `--help` Show help message
-	- This will show the help message
+    - This will show the help message
+
+- `--version` Show version
+    - This will show the version of the program
 
 - `--install` Install the service
     - This will copy the binary to `C:\windows\bridge.exe` and register it as a service

gui.c

@@ -4,6 +4,8 @@
 #include <assert.h>
 #include <stdio.h>
 
+#include "resource.h"
+
 /**
  * The entire code could be better written, but at least it works.
  *
@@ -31,7 +33,7 @@ VOID HandleStartButton(BOOL Silent)
 		SetWindowText(item, "Do you want to start, install or remove the bridge?");
 		RedrawWindow(item, NULL, NULL, RDW_ERASE | RDW_INVALIDATE | RDW_FRAME | RDW_ALLCHILDREN);
 		item = GetDlgItem(hwnd, /* Start Button */ 1);
-		Button_SetText(item, "Start");
+		Button_SetText(item, "&Start");
 		EnableWindow(item, FALSE);
 		RedrawWindow(item, NULL, NULL, RDW_ERASE | RDW_INVALIDATE | RDW_FRAME | RDW_ALLCHILDREN);
 
@@ -47,20 +49,6 @@ VOID HandleStartButton(BOOL Silent)
 		return;
 	}
 
-	if (!IsLinux)
-	{
-		hBridge = CreateThread(NULL, 0, (LPTHREAD_START_ROUTINE)CreateBridge,
-							   NULL, 0, NULL);
-
-		HWND item = GetDlgItem(hwnd, /* Start Button */ 1);
-		Button_SetText(item, "Stop");
-		item = GetDlgItem(hwnd, 4);
-		SetWindowText(item, "Bridge is running...");
-		IsAlreadyRunning = TRUE;
-		ShowWindow(hwnd, SW_MINIMIZE);
-		return;
-	}
-
 	SC_HANDLE hSCManager = OpenSCManager(NULL, NULL, SC_MANAGER_ALL_ACCESS);
 	if (hSCManager == NULL)
 	{
@@ -72,7 +60,7 @@ VOID HandleStartButton(BOOL Silent)
 									   SERVICE_QUERY_CONFIG | SERVICE_CHANGE_CONFIG | SERVICE_START);
 	if (schService == NULL)
 	{
-		// print("OpenService failed: %s\n", GetErrorMessage());
+		print("Service doesn't exist: %s\n", GetErrorMessage());
 
 		/* Service doesn't exist; running without any service */
 
@@ -80,7 +68,7 @@ VOID HandleStartButton(BOOL Silent)
 							   NULL, 0, NULL);
 
 		HWND item = GetDlgItem(hwnd, /* Start Button */ 1);
-		Button_SetText(item, "Stop");
+		Button_SetText(item, "&Stop");
 		item = GetDlgItem(hwnd, 4);
 		SetWindowText(item, "Bridge is running...");
 		IsAlreadyRunning = TRUE;
@@ -115,6 +103,7 @@ VOID HandleStartButton(BOOL Silent)
 	CloseServiceHandle(hSCManager);
 	if (Silent == FALSE)
 		MessageBox(NULL, "Bridge service started successfully", "Info", MB_OK);
+	print("Bridge service started successfully\n");
 }
 
 VOID HandleInstallButton()
@@ -135,6 +124,44 @@ VOID HandleRemoveButton()
 	ExitProcess(0);
 }
 
+void ShowLicenseDialog()
+{
+	HMODULE hModule = GetModuleHandle(NULL);
+	HRSRC hRes = FindResource(hModule, MAKEINTRESOURCE(IDR_LICENSE_TXT), RT_RCDATA);
+	if (!hRes)
+	{
+		MessageBox(NULL, "Resource not found", "Error", MB_OK | MB_ICONERROR);
+		return;
+	}
+
+	HGLOBAL hResData = LoadResource(NULL, hRes);
+	if (!hResData)
+	{
+		MessageBox(NULL, "Resource failed to load", "Error", MB_OK | MB_ICONERROR);
+		return;
+	}
+
+	DWORD resSize = SizeofResource(NULL, hRes);
+	void *pRes = LockResource(hResData);
+	if (!pRes)
+	{
+		MessageBox(NULL, "Resource failed to lock", "Error", MB_OK | MB_ICONERROR);
+		return;
+	}
+
+	char *licenseText = (char *)malloc(resSize + 1);
+	if (!licenseText)
+	{
+		MessageBox(NULL, "Memory allocation failed", "Error", MB_OK | MB_ICONERROR);
+		return;
+	}
+
+	memcpy(licenseText, pRes, resSize);
+	licenseText[resSize] = '\0';
+	MessageBoxA(hwnd, licenseText, "About", MB_OK);
+	free(licenseText);
+}
+
 LRESULT CALLBACK WndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
 {
 	switch (msg)
@@ -152,6 +179,26 @@ LRESULT CALLBACK WndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
 		case 3:
 			HandleRemoveButton();
 			break;
+		case IDM_VIEW_LOG:
+			ShellExecute(NULL, "open", "C:\\windows\\notepad.exe", "C:\\windows\\logs\\bridge.log", NULL, SW_SHOW);
+			break;
+		case IDM_HELP_DOCUMENTATION:
+			ShellExecute(NULL, "open", "https://enderice2.github.io/rpc-bridge/index.html", NULL, NULL, SW_SHOWNORMAL);
+			break;
+		case IDM_HELP_LICENSE:
+			ShowLicenseDialog();
+			break;
+		case IDM_HELP_ABOUT:
+		{
+			char msg[256];
+			sprintf(msg, "rpc-bridge v%s\n\n"
+						 "Simple bridge that allows you to use Discord Rich Presence with Wine games/software.\n\n"
+						 "Created by EnderIce2\n\n"
+						 "Licensed under the MIT License",
+					VER_VERSION_STR);
+			MessageBox(NULL, msg, "About", MB_OK);
+			break;
+		}
 		default:
 			break;
 		}
@@ -178,9 +225,9 @@ LRESULT CALLBACK WndProc(HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam)
 
 VOID SetButtonStyles(INT *btnStartStyle, INT *btnRemoveStyle, INT *btnInstallStyle)
 {
-	*btnStartStyle = WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON;
-	*btnRemoveStyle = WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON;
-	*btnInstallStyle = WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON;
+	*btnStartStyle = WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON | WS_TABSTOP;
+	*btnRemoveStyle = WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON | WS_TABSTOP;
+	*btnInstallStyle = WS_CHILD | WS_VISIBLE | BS_PUSHBUTTON | WS_TABSTOP;
 
 	// if (!IsLinux)
 	// {
@@ -254,21 +301,24 @@ int WINAPI __WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
 								0, 15, 400, 25,
 								hwnd, (HMENU)4, hInstance, NULL);
 
-	HWND hbtn1 = CreateWindow("BUTTON", "Start",
+	HWND hbtn1 = CreateWindow("BUTTON", "&Start",
 							  btnStartStyle,
 							  40, 60, 100, 30,
 							  hwnd, (HMENU)1, hInstance, NULL);
 
-	HWND hbtn2 = CreateWindow("BUTTON", "Install",
+	HWND hbtn2 = CreateWindow("BUTTON", "&Install",
 							  btnInstallStyle,
 							  150, 60, 100, 30,
 							  hwnd, (HMENU)2, hInstance, NULL);
 
-	HWND hbtn3 = CreateWindow("BUTTON", "Remove",
+	HWND hbtn3 = CreateWindow("BUTTON", "&Remove",
 							  btnRemoveStyle,
 							  260, 60, 100, 30,
 							  hwnd, (HMENU)3, hInstance, NULL);
 
+	HMENU hMenu = LoadMenu(hInstance, MAKEINTRESOURCE(IDR_MAINMENU));
+	SetMenu(hwnd, hMenu);
+
 	HDC hDC = GetDC(hwnd);
 	int nHeight = -MulDiv(11, GetDeviceCaps(hDC, LOGPIXELSY), 72);
 
@@ -289,8 +339,11 @@ int WINAPI __WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
 	MSG msg;
 	while (GetMessage(&msg, NULL, 0, 0) > 0)
 	{
-		TranslateMessage(&msg);
-		DispatchMessage(&msg);
+		if (!IsDialogMessage(hwnd, &msg))
+		{
+			TranslateMessage(&msg);
+			DispatchMessage(&msg);
+		}
 	}
 	return msg.wParam;
 }

main.c

@@ -3,6 +3,8 @@
 #include <assert.h>
 #include <stdio.h>
 
+#include "resource.h"
+
 FILE *g_logFile = NULL;
 BOOL RunningAsService = FALSE;
 
@@ -72,14 +74,11 @@ void DetectWine()
 	{
 		int result = MessageBox(NULL, "This program is designed for Linux and macOS only!\nDo you want to proceed?",
 								NULL, MB_YESNO | MB_ICONQUESTION);
-		if (result == IDYES)
-			return;
-		else if (result == IDNO)
+		if (result == IDNO)
 			ExitProcess(1);
 	}
 
 	IsLinux = strcmp(__sysname, "Linux") == 0;
-	printf("Running on %s\n", __sysname);
 }
 
 void print(char const *fmt, ...)
@@ -272,32 +271,40 @@ void HandleArguments(int argc, char *argv[])
 		CreateBridge();
 		ExitProcess(0);
 	}
+	else if (strcmp(argv[1], "--version") == 0)
+	{
+		printf("%s\n", VER_VERSION_STR);
+		ExitProcess(0);
+	}
 	else if (strcmp(argv[1], "--help") == 0)
 	{
-		printf("Usage:\n");
-		printf("  %s [args]\n\n", argv[0]);
-
-		printf("Arguments:\n");
-		printf("  --help         Show this help\n\n");
-
-		printf("  --install      Install service\n");
-		printf("        This will copy the binary to C:\\windows\\bridge.exe and register it as a service\n\n");
-
-		printf("  --uninstall    Uninstall service\n");
-		printf("        This will remove the service and delete C:\\windows\\bridge.exe\n\n");
-
-		printf("  --steam        Reserved for Steam\n");
-		printf("        This will start the service and exit (used with bridge.sh)\n\n");
-
-		printf("  --no-service   Do not run as service\n");
-		printf("        (only for --steam)\n\n");
-
-		printf("  --service      Reserved for service\n\n");
-
-		printf("  --rpc <dir>    Set RPC_PATH environment variable\n");
-		printf("        This is used to specify the directory where 'discord-ipc-0' is located\n\n");
-
-		printf("Note: If no arguments are provided, the GUI will be shown instead\n");
+		printf("Usage:\n"
+			   "  %s [args]\n"
+			   "\n"
+			   "Arguments:\n"
+			   "  --help         Show this help\n"
+			   "\n"
+			   "  --version      Show version\n"
+			   "\n"
+			   "  --install      Install service\n"
+			   "        This will copy the binary to C:\\windows\\bridge.exe and register it as a service\n"
+			   "\n"
+			   "  --uninstall    Uninstall service\n"
+			   "        This will remove the service and delete C:\\windows\\bridge.exe\n"
+			   "\n"
+			   "  --steam        Reserved for Steam\n"
+			   "        This will start the service and exit (used with bridge.sh)\n"
+			   "\n"
+			   "  --no-service   Do not run as service\n"
+			   "        (only for --steam)\n"
+			   "\n"
+			   "  --service      Reserved for service\n"
+			   "\n"
+			   "  --rpc <dir>    Set RPC_PATH environment variable\n"
+			   "        This is used to specify the directory where 'discord-ipc-0' is located\n"
+			   "\n"
+			   "Note: If no arguments are provided, the GUI will be shown instead\n",
+			   argv[0]);
 		ExitProcess(0);
 	}
 }

resource.h

@@ -0,0 +1,9 @@
+#define IDR_MAINMENU 101
+#define IDR_LICENSE_TXT 102
+#define IDM_HELP_DOCUMENTATION 40001
+#define IDM_HELP_LICENSE 40002
+#define IDM_HELP_ABOUT 40003
+#define IDM_VIEW_LOG 40004
+
+#define VER_VERSION 1, 1, 0, 0
+#define VER_VERSION_STR "1.1\0"