From 1d1d19ed30309b39fc5e43c830cabb4cdd004d07 Mon Sep 17 00:00:00 2001 From: Randy Palamar Date: Fri, 5 Dec 2025 12:05:32 -0700 Subject: mark all functions in headers with VIS_EXPORT or VIS_INTERNAL if vis actually wants to be a library exported symbols may need mark up depending on the platform (eg. __declspec(dllexport)). This needs to be hidden behind a macro because the way you export is not the same on every platform. I did this based on the assumption that vis.h was supposed to be the only interface to the "vis" library. Since nobody actually uses vis as a library I have no idea if this is actually correct. Anyway marking up all prototypes like this allows for one to convert all functions to static if a single translation unit is used by inserting at the start: #define VIS_INTERNAL static #define VIS_EXPORT static --- vis-subprocess.h | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'vis-subprocess.h') diff --git a/vis-subprocess.h b/vis-subprocess.h index 79a043e..05f4dfe 100644 --- a/vis-subprocess.h +++ b/vis-subprocess.h @@ -23,9 +23,9 @@ struct Process { typedef enum { STDOUT, STDERR, SIGNAL, EXIT } ResponseType; -Process *vis_process_communicate(Vis *, const char *command, const char *name, - Invalidator **invalidator); -int vis_process_before_tick(fd_set *); -void vis_process_tick(Vis *, fd_set *); -void vis_process_waitall(Vis *); +VIS_INTERNAL Process *vis_process_communicate(Vis *, const char *command, const char *name, + Invalidator **invalidator); +VIS_INTERNAL int vis_process_before_tick(fd_set *); +VIS_INTERNAL void vis_process_tick(Vis *, fd_set *); +VIS_INTERNAL void vis_process_waitall(Vis *); #endif -- cgit v1.2.3