https://gitlab.synchro.net/main/sbbs/-/commit/5ea2eeb311f22a1e5f826882
Modified Files:
src/xpdev/dirwrap.c
Log Message:
If delfiles() is called with a NULL spec, assume "all files" (*)

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/d9d140af60f18552e07150de
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Fix disksize/space functions for Win32

Broken 11 months ago with commit ae44ab15: the unit argument was ignored (e.g. converting to kibibytes) and the free/total disk size/space checks were reversed.

Reported by Max (WESTLINE) via usage of JS properties: system.freediskspace and system.freediskspacek.

The JS global functions dir_freespace() and disk_size() were also broken due to this bug.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/095858e5c974d1e6265f0b65
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Set size to MAX_PATH+1 when allocating.

On Win32, size is explcitly "ignored" when target is NULL, so we
should set it to the size of the allocation.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/cb0dbdaf0928ea962e5b8c9d
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Fix glob() for Win64

_fintfirst() is declared in the Mingw-w64 headers as an intptr_t,
which tracks. Can't find _findfirst on MSDN, so that's good enough
for me... let's see what the pipeline says.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/bf757d6952f7ac30b9f0c819
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Not every platform has GLOB_ONLYDIR support

... kind of as I suspected

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/abac966eec71a2f3af83ae6c
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Optimize mkpath()

If the path already exists (is a directory), don't need to check each path element for existence.

This should help with the speed of saving of large file.ini (Nelgin's) when a lot of directories have "Check for File Existence" set to "Yes".

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/5e025ef3f92105834d4c7178
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Fix bug in recursive use of delfiles(), not deleting all subdirs/files

Looks like this feature (commit d661427e13) never really worked correctly
since it counted the files removed from each sub-dir and then stopped deleting when the count reached the number of files in the base directory. This was
done to accommodate the 'keep' feature (part of previous commits).
So make 'keep' check conditional on it being non-zero and just don't ever use
a non-zero keep value with a recursive delete and we should be good! :-)

This fixes issue #841

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/c82eff12b58be6dca245c599
Modified Files:
src/xpdev/dirwrap.c
Log Message:
Comment out access() call in Windows version of getfilecase()

I'm hoping this might help resolve instances where fexistcase() says a file does not exist, but a subsequent call to fexist() says it does.

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net

https://gitlab.synchro.net/main/sbbs/-/commit/e7cfae3e9ad94036156dd0ba
Modified Files:
src/xpdev/dirwrap.c
Log Message:
A little style clean-up, reduced copy/paste, no functional changes

Reduce the cognitive dissonance

---
■ Synchronet ■ Vertrauen ■ Home of Synchronet ■ [vert/cvs/bbs].synchro.net