For some reason, since the latest MSYS2 update on my Windows system, I got compiler errors that `g_stat()` requires a `GStatBuf` type as second argument.
According to the docs, https://docs.gtk.org/glib/func.stat.html, this is correct and in Geany we also use `GStatBuf` in all places. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/1353
-- Commit Summary --
* ProjectOrganizer, Workbench: Use GStatBuf for "g_stat()" calls
-- File Changes --
M projectorganizer/src/prjorg-project.c (2) M workbench/src/tm_control.c (2)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/1353.patch https://github.com/geany/geany-plugins/pull/1353.diff
For some reason, since the latest MSYS2 update on my Windows system, I got compiler errors that g_stat() requires a GStatBuf type as second argument.
On windows there are many MS `stat` calls and `struct stat` is a structure that changes depending on platform, compiler, target os, wind direction etc and in some cases may not match `g_stat` which calls only one of the many MS `stat` calls. Perhaps they could have made `g_stat()` call the right function for the definition of `struct stat` but probably it was easier to define `GStatBuf` to match the MS call made by `g_stat()`. Maybe which MS call `g_stat()` uses changed, it was 32 bit times IIRC and maybe they finally switched to 64 bit ones, so its good the compiler now enforces it.
LGBI
@techee approved this pull request.
LGTM, thanks!
Merged #1353 into master.
github-comments@lists.geany.org