core.system.Sys.printf() implemented (depends on libc version) and used

Author: denizzzka

druntime/src/core/gc/config.d

@@ -9,6 +9,7 @@ module core.gc.config;
 
 import core.internal.parseoptions;
 import core.stdc.stdio : printf;
+import core.system : Sys;
 import core.system.opt : Opt;
 
 alias Config = Opt.GcConfig;
@@ -57,21 +58,21 @@ struct ConfigT()
     {
         import core.gc.registry : registeredGCFactories;
 
-        printf("GC options are specified as white space separated assignments:
+        Sys.printf("GC options are specified as white space separated assignments:
     disable:0|1    - start disabled (%d)
     fork:0|1       - set fork behaviour (%d)
     profile:0|1|2  - enable profiling with summary when terminating program (%d)
-    gc:".ptr, disable, fork, profile);
+    gc:", disable, fork, profile);
         foreach (i, entry; registeredGCFactories)
         {
-            if (i) printf("|");
+            if (i) Sys.print("|");
             printf("%.*s", cast(int) entry.name.length, entry.name.ptr);
         }
         auto _initReserve = initReserve.bytes2prettyStruct;
         auto _minPoolSize = minPoolSize.bytes2prettyStruct;
         auto _maxPoolSize = maxPoolSize.bytes2prettyStruct;
         auto _incPoolSize = incPoolSize.bytes2prettyStruct;
-        printf(" - select gc implementation (default = conservative)
+        Sys.printf(" - select gc implementation (default = conservative)
 
     initReserve:N  - initial memory to reserve in MB (%lld%c)
     minPoolSize:N  - initial and minimum pool size in MB (%lld%c)
@@ -82,7 +83,7 @@ struct ConfigT()
     cleanup:none|collect|finalize - how to treat live objects when terminating (collect)
 
     Memory-related values can use B, K, M or G suffixes.
-".ptr,
+",
                _initReserve.v, _initReserve.u,
                _minPoolSize.v, _minPoolSize.u,
                _maxPoolSize.v, _maxPoolSize.u,

druntime/src/core/system/package.d

@@ -33,30 +33,12 @@ struct Sys
             Sys.abort();
     }
 
-    //~ void print(T)(scope const char[] str, T vals) nothrow @nogc
-    //~ {
-        //~ // Assume str is null-terminated string
-        //~ auto r = fputs(str.ptr, stdout);
-        //~ if(r < 0)
-            //~ Sys.abort();
-    //~ }
-
-    //~ alias format = snprintf
-}
-
-// Special case: we don't have resources to implement mature formatting
-// functionality for now just for the sake of using it at debugging
-debug
-{
-    static import core.stdc.stdio;
-
-    alias sprintf = core.stdc.stdio.sprintf;
-    alias snprintf = core.stdc.stdio.snprintf;
-
-    void printf(T...)(scope const(char*) fmt, T vals) nothrow @nogc
+    /// C-formatted print
+    static void printf(T...)(scope const char[] fmt, T vals) nothrow @nogc
     {
-        int r = core.stdc.stdio.printf(fmt, vals);
+        static assert(T.length > 0);
 
+        int r = core.stdc.stdio.printf(fmt.ptr, vals);
         if(r < fmt.length)
             Sys.abort();
     }