PictView thumbnails not rendered for long paths (>260 chars)

[0xeb]

Split from #24. Image thumbnails are not rendered for files with paths >260 chars on UNC/WSL shares.

Root cause: PictView plugin likely uses CreateFileW/GetFileAttributesW without \?\ prefix, same pattern fixed in #24 for the internal viewer.

Deferred until PictView is added to the CMake build. The fix approach is the same: replace raw Win32 calls with SalLPCreateFile/SalLPGetFileAttributes from widepath.h.

Original report: #24

[melloware]

@0xeb is there a change the pictview plugin can be rewritten to not use that proprietary DLL? I love this plugin I use it practically every day for screenshots and for clipping images etc. I would love it if it was just part your build.

[0xeb]

Heads-up: Sally v1.0.14 (https://github.com/0xeb/sally/releases/tag/v1.0.14) extends long-path support across the file-operation pipeline (worker, copy/move, external launchers, panel/Find/history wide caches). PictView's thumbnail render path was not directly touched in this round, so this issue stays open, but the underlying long-path plumbing it depends on is now more robust.

If you can retest on v1.0.14 and either confirm the symptom changed or paste a fresh repro with a current path length, that would help triage the remaining PictView-specific work.

[melloware]

@0xeb thanks the long file names have been a pain when i go to Delete a folder in SAlamander and it tells me "path is too long" :)

[0xeb]

@melloware please try again and if it repros, give me an example path. So I just try to go to a share and have a very long path??

[melloware]

OK it doesn't alway happen but i will try and get a reproducer. Sometimes its when I using an NPM project and something in /node_modules folder must be very long. I then have to go to Command Prompt and remove the directory.