newconfig: move some includes and macros to .c

and make internal function static.
metalefty · Aug 20, 2024 · d194449 · d194449
1 parent 73bdc5d
commit d194449
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 13 deletions.
diff --git a/xrdp/xrdp_newconfig.c b/xrdp/xrdp_newconfig.c
@@ -36,15 +36,20 @@
 #include "os_calls.h"
 #include "parse.h"
 #include "toml.h"
+#include "ms-rdpbcgr.h"
 #include "xrdp_newconfig.h"
+#include "string_calls.h"
+
+#define GFX_CONF XRDP_CFG_PATH "/gfx.toml"
+#define NCLOG(log_level, args...) LOG(log_level, "NewConfig: " args)
 
 #define X264_DEFAULT_PRESET  "ultrafast"
 #define X264_DEFAULT_TUNE    "zerolatency"
 #define X264_DEFAULT_PROFILE "main"
 #define X264_DEFAULT_FPS_NUM 24
 #define X264_DEFAULT_FPS_DEN 1
 
-int
+static int
 nc_load_gfx_x264_ct(toml_table_t *tfile, const int connection_type,
                     struct xrdp_nc_gfx_x264_param *param)
 {

diff --git a/xrdp/xrdp_newconfig.h b/xrdp/xrdp_newconfig.h
@@ -19,17 +19,12 @@
 #ifndef _XRDP_NEWCONFIG_H_
 #define _XRDP_NEWCONFIG_H_
 
-#include "toml.h"
-#include "string_calls.h"
-#include "ms-rdpbcgr.h"
+struct toml_table_t;
+typedef struct toml_table_t toml_table_t;
 
-#if !defined(NUM_CONNECTION_TYPES)
-#define NUM_CONNECTION_TYPES CONNECTION_TYPE_AUTODETECT
-#endif
-
-#define GFX_CONF XRDP_CFG_PATH "/gfx.toml"
+/* The number of connection types in MS-RDPBCGR 2.2.1.3.2 */
+#define NUM_CONNECTION_TYPES 7
 
-#define NCLOG(log_level, args...) LOG(log_level, "NewConfig: " args)
 
 /* nc stands for new config */
 struct xrdp_nc_gfx_x264_param
@@ -62,9 +57,6 @@ static const char *const rdpbcgr_connection_type_names[] =
     0
 };
 
-
-int nc_load_gfx_x264_ct(toml_table_t *tfile, const int connection_type,
-                        struct xrdp_nc_gfx_x264_param *param);
 int nc_load_gfx(const char *filename, struct xrdp_nc_gfx *config);
 
 #endif