Rework OS abstractions

- As much as possible avoid dragging system headers into translation
  units. This should avoid namespace pollution and, hopefully, speed up
  builds a little bit.

- Avoid leaning on the UCRT so much on Windows - prefer native win32
  calls and native file handles except where doing so is inconvenient
  (in particular, for stat(), which we might try and replace later).

- Also, switch from SystemFunction036 to ProcessPrng on Windows. This
  requires us to generate a stub for bcryptprimitives.dll because
  Microsoft haven't bothered to provide a link library, but the function
  is better-documented and seems to be a more direct under-the-hood call
  as well. Apparently it's what's used by the major web browsers these
  days, which seems like a good indication it's stable and trusted.

- Lastly, remove a bunch of functions and macros and stuff that weren't
  actually being used. It seems good to try and keep the scope of
  OS-dependent stuff relatively contained and only add to it when
  actually required.

1 files changed, 11 insertions, 11 deletions

diff --git a/src/build/cmeta.c b/src/build/cmeta.c
index 0c9b3ca..8a2416d 100644
--- a/src/build/cmeta.c
+++ b/src/build/cmeta.c
@@ -1,5 +1,5 @@
 /*
- * Copyright © 2023 Michael Smith <mikesmiffy128@gmail.com>
+ * Copyright © 2024 Michael Smith <mikesmiffy128@gmail.com>
  *
  * Permission to use, copy, modify, and/or distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -102,15 +102,15 @@ static void die(const char *s) {
 	exit(100);
 }
 
-static char *readsource(const os_char *f) {
-	int fd = os_open(f, O_RDONLY);
-	if (fd == -1) return 0;
+static char *readsource(const os_char *path) {
+	int f = os_open_read(path);
+	if (f == -1) return 0;
 	uint bufsz = 8192;
 	char *buf = malloc(bufsz);
 	if (!buf) die("couldn't allocate memory");
 	int nread;
 	int off = 0;
-	while ((nread = read(fd, buf + off, bufsz - off)) > 0) {
+	while ((nread = os_read(f, buf + off, bufsz - off)) > 0) {
 		off += nread;
 		if (off == bufsz) {
 			bufsz *= 2;
@@ -122,24 +122,24 @@ static char *readsource(const os_char *f) {
 	}
 	if (nread == -1) die("couldn't read file");
 	buf[off] = 0;
-	close(fd);
+	os_close(f);
 	return buf;
 }
 
 // as per cmeta.h this is totally opaque; it's actually just a Token in disguise
 struct cmeta;
 
-const struct cmeta *cmeta_loadfile(const os_char *f) {
-	char *buf = readsource(f);
+const struct cmeta *cmeta_loadfile(const os_char *path) {
+	char *buf = readsource(path);
 	if (!buf) return 0;
 #ifdef _WIN32
-	char *realname = malloc(wcslen(f) + 1);
+	char *realname = malloc(wcslen(path) + 1);
 	if (!realname) die("couldn't allocate memory");
 	// XXX: being lazy about Unicode right now; a general purpose tool should
 	// implement WTF8 or something. SST itself doesn't have any unicode paths
 	// though, so don't really care as much.
-	*realname = *f;
-	for (const ushort *p = f + 1; p[-1]; ++p) realname[p - f] = *p;
+	*realname = *path;
+	for (const ushort *p = path + 1; p[-1]; ++p) realname[p - path] = *p;
 #else
 	const char *realname = f;
 #endif