diff --git a/changelog.md b/changelog.md index 70ebbac0978d9..840a19b51bb98 100644 --- a/changelog.md +++ b/changelog.md @@ -14,6 +14,7 @@ - `std/assertions` - `std/formatfloat` - `std/objectdollar` + - `std/widestrs` In the future, these definitions will be removed from the `system` module, and their respective modules will have to be imported to use them. diff --git a/lib/pure/asyncfile.nim b/lib/pure/asyncfile.nim index 9cc9f5b4895f8..217dca6d94100 100644 --- a/lib/pure/asyncfile.nim +++ b/lib/pure/asyncfile.nim @@ -26,6 +26,8 @@ import asyncdispatch, os when defined(nimPreviewSlimSystem): import std/[assertions, syncio] + when defined(windows): + import std/widestrs # TODO: Fix duplication introduced by PR #4683. diff --git a/lib/pure/browsers.nim b/lib/pure/browsers.nim index c36e31b118f05..b95b37a1d1d49 100644 --- a/lib/pure/browsers.nim +++ b/lib/pure/browsers.nim @@ -21,6 +21,8 @@ when defined(nimPreviewSlimSystem): when defined(windows): import winlean + when useWinUnicode and defined(nimPreviewSlimSystem): + import std/widestrs from os import absolutePath else: import os diff --git a/lib/pure/includes/osenv.nim b/lib/pure/includes/osenv.nim index b7200a8e2a287..a1d906519fe6a 100644 --- a/lib/pure/includes/osenv.nim +++ b/lib/pure/includes/osenv.nim @@ -43,6 +43,8 @@ when not defined(nimscript): else: when defined(windows): + when defined(nimPreviewSlimSystem): + import std/widestrs proc c_putenv(envstring: cstring): cint {.importc: "_putenv", header: "".} from std/private/win_setenv import setEnvImpl proc c_wgetenv(varname: WideCString): WideCString {.importc: "_wgetenv", diff --git a/lib/pure/includes/oserr.nim b/lib/pure/includes/oserr.nim index a6eba84ba10da..c58fdb22c4272 100644 --- a/lib/pure/includes/oserr.nim +++ b/lib/pure/includes/oserr.nim @@ -11,6 +11,8 @@ when not defined(nimscript): when defined(windows): import winlean + when useWinUnicode and defined(nimPreviewSlimSystem): + import std/widestrs proc `==`*(err1, err2: OSErrorCode): bool {.borrow.} proc `$`*(err: OSErrorCode): string {.borrow.} diff --git a/lib/pure/memfiles.nim b/lib/pure/memfiles.nim index a952933eda4c9..d0877b311b30e 100644 --- a/lib/pure/memfiles.nim +++ b/lib/pure/memfiles.nim @@ -17,6 +17,8 @@ when defined(windows): import winlean + when useWinUnicode and defined(nimPreviewSlimSystem): + import std/widestrs elif defined(posix): import posix else: diff --git a/lib/pure/os.nim b/lib/pure/os.nim index a39fcaf46995e..b1292a648bba8 100644 --- a/lib/pure/os.nim +++ b/lib/pure/os.nim @@ -968,6 +968,9 @@ proc getCacheDir*(app: string): string = when defined(windows): type DWORD = uint32 + when defined(nimPreviewSlimSystem): + import std/widestrs + proc getTempPath( nBufferLength: DWORD, lpBuffer: WideCString ): DWORD {.stdcall, dynlib: "kernel32.dll", importc: "GetTempPathW".} = diff --git a/lib/pure/osproc.nim b/lib/pure/osproc.nim index a0079cf95323d..5e130b47a5e57 100644 --- a/lib/pure/osproc.nim +++ b/lib/pure/osproc.nim @@ -33,6 +33,8 @@ when defined(linux) and defined(useClone): when defined(nimPreviewSlimSystem): import std/[syncio, assertions] + when defined(windows): + import std/widestrs type diff --git a/lib/std/envvars.nim b/lib/std/envvars.nim index d7706c17d1dbc..9748cd2ad2ffa 100644 --- a/lib/std/envvars.nim +++ b/lib/std/envvars.nim @@ -61,6 +61,8 @@ when not defined(nimscript): proc c_putenv(envstring: cstring): cint {.importc: "_putenv", header: "".} from std/private/win_setenv import setEnvImpl import winlean + when defined(nimPreviewSlimSystem): + import std/widestrs proc c_wgetenv(varname: WideCString): WideCString {.importc: "_wgetenv", header: "".} proc getEnvImpl(env: cstring): WideCString = c_wgetenv(env.newWideCString) diff --git a/lib/std/oserrors.nim b/lib/std/oserrors.nim index 9c2649eab8cc8..8d06c41dadac7 100644 --- a/lib/std/oserrors.nim +++ b/lib/std/oserrors.nim @@ -16,6 +16,8 @@ type when not defined(nimscript): when defined(windows): import winlean + when defined(nimPreviewSlimSystem): + import std/widestrs else: var errno {.importc, header: "".}: cint diff --git a/lib/std/private/win_setenv.nim b/lib/std/private/win_setenv.nim index 89bb0421fd9e3..9315f3e7ecff5 100644 --- a/lib/std/private/win_setenv.nim +++ b/lib/std/private/win_setenv.nim @@ -23,6 +23,9 @@ check errno_t vs cint when not defined(windows): discard else: + when defined(nimPreviewSlimSystem): + import std/widestrs + type wchar_t {.importc: "wchar_t".} = int16 proc setEnvironmentVariableW*(lpName, lpValue: WideCString): int32 {. diff --git a/lib/std/syncio.nim b/lib/std/syncio.nim index fc132bba7d28c..dadad97918304 100644 --- a/lib/std/syncio.nim +++ b/lib/std/syncio.nim @@ -12,6 +12,8 @@ include system/inclrtl import std/private/since import std/formatfloat +when defined(windows): + import std/widestrs # ----------------- IO Part ------------------------------------------------ type diff --git a/lib/std/tempfiles.nim b/lib/std/tempfiles.nim index ee42f8a5c99fd..1160aaaad54ed 100644 --- a/lib/std/tempfiles.nim +++ b/lib/std/tempfiles.nim @@ -30,6 +30,8 @@ const when defined(windows): import winlean + when defined(nimPreviewSlimSystem): + import std/widestrs var O_RDWR {.importc: "_O_RDWR", header: "".}: cint diff --git a/lib/system/widestrs.nim b/lib/std/widestrs.nim similarity index 98% rename from lib/system/widestrs.nim rename to lib/std/widestrs.nim index bb348fd6776a3..7df4004d195f3 100644 --- a/lib/system/widestrs.nim +++ b/lib/std/widestrs.nim @@ -7,8 +7,7 @@ # distribution, for details about the copyright. # -# Nim support for C/C++'s `wide strings`:idx:. This is part of the system -# module! Do not import it directly! +# Nim support for C/C++'s `wide strings`:idx:. #when not declared(ThisIsSystem): # {.error: "You must not import this module explicitly".} diff --git a/lib/system.nim b/lib/system.nim index daee96c1c5ca7..fd31cf3da3743 100644 --- a/lib/system.nim +++ b/lib/system.nim @@ -2655,8 +2655,9 @@ when defined(genode): # and return to thread entrypoint. -import system/widestrs -export widestrs +when not defined(nimPreviewSlimSystem): + import std/widestrs + export widestrs when notJSnotNims: when defined(windows) and compileOption("threads"): diff --git a/lib/system_overview.rst b/lib/system_overview.rst index 1580693753ece..b338845d68b8a 100644 --- a/lib/system_overview.rst +++ b/lib/system_overview.rst @@ -8,7 +8,6 @@ is in separate files: * `exceptions `_ * `assertions `_ * `dollars `_ -* `widestrs `_ * `ctypes `_ diff --git a/lib/windows/registry.nim b/lib/windows/registry.nim index 44e4e442087a9..6abc41a89329e 100644 --- a/lib/windows/registry.nim +++ b/lib/windows/registry.nim @@ -11,6 +11,9 @@ import winlean, os +when defined(nimPreviewSlimSystem): + import std/widestrs + type HKEY* = uint diff --git a/lib/windows/winlean.nim b/lib/windows/winlean.nim index 91b0d18c10ea5..93ab3dd2ebcca 100644 --- a/lib/windows/winlean.nim +++ b/lib/windows/winlean.nim @@ -19,6 +19,7 @@ when defined(nimHasStyleChecks): when defined(nimPreviewSlimSystem): from std/syncio import FileHandle + import std/widestrs const useWinUnicode* = not defined(useWinAnsi) diff --git a/lib/wrappers/odbcsql.nim b/lib/wrappers/odbcsql.nim index 4e37e1a174676..6b300c5ffeba8 100644 --- a/lib/wrappers/odbcsql.nim +++ b/lib/wrappers/odbcsql.nim @@ -7,6 +7,9 @@ # distribution, for details about the copyright. # +when defined(nimPreviewSlimSystem): + import std/widestrs + when not defined(ODBCVER): const ODBCVER = 0x0351 ## define ODBC version 3.51 by default diff --git a/tests/arc/t19862.nim b/tests/arc/t19862.nim index f7146ec2689d4..6d3f57692631e 100644 --- a/tests/arc/t19862.nim +++ b/tests/arc/t19862.nim @@ -2,6 +2,8 @@ discard """ matrix: "--gc:refc; --gc:arc" """ +import std/widestrs + # bug #19862 type NewString = object diff --git a/tests/ccgbugs/tcgbug.nim b/tests/ccgbugs/tcgbug.nim index 0fe4b8852527b..14ed390d40520 100644 --- a/tests/ccgbugs/tcgbug.nim +++ b/tests/ccgbugs/tcgbug.nim @@ -24,6 +24,7 @@ q(a) # bug #914 when defined(windows): + import std/widestrs var x = newWideCString("Hello") echo "success" diff --git a/tests/effects/tstrict_funcs_imports.nim b/tests/effects/tstrict_funcs_imports.nim index 264771a1ad479..eb83bb69b4bb4 100644 --- a/tests/effects/tstrict_funcs_imports.nim +++ b/tests/effects/tstrict_funcs_imports.nim @@ -7,6 +7,15 @@ discard """ when defined(linux): import linenoise +when defined(nimPreviewSlimSystem): + import std/[ + assertions, + formatfloat, + objectdollar, + syncio, + widestrs, + ] + import algorithm, asyncdispatch, diff --git a/tests/js/tstdlib_imports.nim b/tests/js/tstdlib_imports.nim index 30aca7561320d..ed01ff6a415a6 100644 --- a/tests/js/tstdlib_imports.nim +++ b/tests/js/tstdlib_imports.nim @@ -4,6 +4,11 @@ discard """ {.warning[UnusedImport]: off.} +when defined(nimPreviewSlimSystem): + import std/[ + syncio, assertions, formatfloat, objectdollar, widestrs + ] + import std/[ # Core: bitops, typetraits, lenientops, macros, volatile, typeinfo, diff --git a/tests/stdlib/t15663.nim b/tests/stdlib/t15663.nim index 1ad5677fdf1b8..8e8bfd9a826f7 100644 --- a/tests/stdlib/t15663.nim +++ b/tests/stdlib/t15663.nim @@ -3,5 +3,7 @@ discard """ output: "Test" """ +import std/widestrs + let ws = newWideCString("Test") -echo ws \ No newline at end of file +echo ws diff --git a/tests/stdlib/tenvvars.nim b/tests/stdlib/tenvvars.nim index 47c1ad24aba80..2d084f71d8a46 100644 --- a/tests/stdlib/tenvvars.nim +++ b/tests/stdlib/tenvvars.nim @@ -49,6 +49,7 @@ template main = main() when defined(windows): + import std/widestrs proc c_wgetenv(env: WideCString): WideCString {.importc: "_wgetenv", header: "".} proc c_getenv(env: cstring): cstring {.importc: "getenv", header: "".} diff --git a/tests/stdlib/tosenv.nim b/tests/stdlib/tosenv.nim index f7b3bb9d60b3a..365edc8c2f4e6 100644 --- a/tests/stdlib/tosenv.nim +++ b/tests/stdlib/tosenv.nim @@ -50,6 +50,7 @@ static: main() main() when defined(windows): + import std/widestrs proc c_wgetenv(env: WideCString): WideCString {.importc: "_wgetenv", header: "".} proc c_getenv(env: cstring): cstring {.importc: "getenv", header: "".} diff --git a/tests/stdlib/twchartoutf8.nim b/tests/stdlib/twchartoutf8.nim index add104b07d1a4..0b6cf696ea7ea 100644 --- a/tests/stdlib/twchartoutf8.nim +++ b/tests/stdlib/twchartoutf8.nim @@ -10,6 +10,7 @@ import std/[syncio, assertions] when not defined(windows): echo "OK" else: + import std/widestrs {.push gcsafe.} const CP_UTF8 = 65001'i32 diff --git a/tests/test_nimscript.nims b/tests/test_nimscript.nims index 5fc77f6e4b3a0..a68e00fe176f5 100644 --- a/tests/test_nimscript.nims +++ b/tests/test_nimscript.nims @@ -6,7 +6,9 @@ from stdtest/specialpaths import buildDir when defined(nimPreviewSlimSystem): - import std/syncio + import std/[ + syncio, assertions, formatfloat, objectdollar, widestrs + ] import std/[ # Core: diff --git a/tools/kochdocs.nim b/tools/kochdocs.nim index 18659825b88c6..ee9c29499bace 100644 --- a/tools/kochdocs.nim +++ b/tools/kochdocs.nim @@ -197,7 +197,6 @@ lib/system/assertions.nim lib/system/iterators.nim lib/system/exceptions.nim lib/system/dollars.nim -lib/system/widestrs.nim lib/system/ctypes.nim """.splitWhitespace()