mirror of
https://github.com/subsurface/subsurface.git
synced 2024-12-01 06:30:26 +00:00
725e4582d9
On Linux and MacOS the subsurface_close_conf() doesn't really close the config file (it flushes writes on MacOS), but on Windows it does actually close the registry hkey. Which is bad, if you change the settings multiple times - we assume that the config file is open the whole time. So add a "subsurface_flush_conf()" function, and call *that* when changing configuration parameters. And call the close function only at the very end. Alternatively, maybe we should just open the config file separately every time. I don't much care, maybe somebody else does. Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
104 lines
2.6 KiB
C
104 lines
2.6 KiB
C
/* windows.c */
|
|
/* implements Windows specific functions */
|
|
#include "display-gtk.h"
|
|
#include <windows.h>
|
|
#define DIVELIST_DEFAULT_FONT "Sans 8"
|
|
|
|
static HKEY hkey;
|
|
|
|
static int get_from_registry(HKEY hkey, const char *key)
|
|
{
|
|
DWORD value;
|
|
DWORD len = 4;
|
|
LONG success;
|
|
|
|
success = RegQueryValueEx(hkey, (LPCTSTR)TEXT(key), NULL, NULL,
|
|
(LPBYTE) &value, (LPDWORD)&len );
|
|
if (success != ERROR_SUCCESS)
|
|
return FALSE; /* that's what happens the first time we start */
|
|
return value;
|
|
}
|
|
|
|
void subsurface_open_conf(void)
|
|
{
|
|
LONG success;
|
|
|
|
success = RegCreateKeyEx(HKEY_CURRENT_USER, (LPCTSTR)TEXT("Software\\subsurface"),
|
|
0L, NULL, REG_OPTION_NON_VOLATILE, KEY_ALL_ACCESS,
|
|
NULL, &hkey, NULL);
|
|
if (success != ERROR_SUCCESS)
|
|
printf("CreateKey Software\\subsurface failed %ld\n", success);
|
|
}
|
|
|
|
void subsurface_set_conf(char *name, pref_type_t type, const void *value)
|
|
{
|
|
/* since we are using the pointer 'value' as both an actual
|
|
* pointer to the string setting and as a way to pass the
|
|
* numbers 0 and 1 to this function for booleans, one of the
|
|
* calls to RegSetValueEx needs to pass &value (when we want
|
|
* to pass the boolean value), the other one passes value (the
|
|
* address of the string. */
|
|
switch (type) {
|
|
case PREF_BOOL:
|
|
/* we simply store the value as DWORD */
|
|
RegSetValueEx(hkey, (LPCTSTR)TEXT(name), 0, REG_DWORD, (const BYTE *)&value, 4);
|
|
break;
|
|
case PREF_STRING:
|
|
RegSetValueEx(hkey, (LPCTSTR)TEXT(name), 0, REG_SZ, (const BYTE *)value, strlen(value));
|
|
}
|
|
}
|
|
|
|
const void *subsurface_get_conf(char *name, pref_type_t type)
|
|
{
|
|
LONG success;
|
|
char *string;
|
|
int len;
|
|
|
|
switch (type) {
|
|
case PREF_BOOL:
|
|
return get_from_registry(hkey, name) ? (void *) 1 : NULL;
|
|
case PREF_STRING:
|
|
string = malloc(80);
|
|
len = 80;
|
|
success = RegQueryValueEx(hkey, (LPCTSTR)TEXT(name), NULL, NULL,
|
|
(LPBYTE) string, (LPDWORD)&len );
|
|
if (success != ERROR_SUCCESS) {
|
|
/* that's what happens the first time we start - just return NULL */
|
|
free(string);
|
|
return NULL;
|
|
}
|
|
return string;
|
|
}
|
|
/* we shouldn't get here */
|
|
return NULL;
|
|
}
|
|
|
|
void subsurface_flush_conf(void)
|
|
{
|
|
/* I wonder if we should even do this - it's apparently very expensive */
|
|
if (RegFlushKey(hkey) != ERROR_SUCCESS)
|
|
printf("RegFlushKey failed \n");
|
|
}
|
|
|
|
void subsurface_close_conf(void)
|
|
{
|
|
RegCloseKey(hkey);
|
|
}
|
|
|
|
const char *subsurface_USB_name()
|
|
{
|
|
return "COM3";
|
|
}
|
|
|
|
const char *subsurface_icon_name()
|
|
{
|
|
return "subsurface.ico";
|
|
}
|
|
|
|
void subsurface_ui_setup(GtkSettings *settings, GtkWidget *menubar,
|
|
GtkWidget *vbox, GtkUIManager *ui_manager)
|
|
{
|
|
if (!divelist_font)
|
|
divelist_font = DIVELIST_DEFAULT_FONT;
|
|
gtk_box_pack_start(GTK_BOX(vbox), menubar, FALSE, FALSE, 0);
|
|
}
|