Menu doesn't find *.lua files *Solved

[otse]

void Menu::FindScripts()

https://github.com/Force67/DawnHook/blob/master/code/dawnhook/Menu.cpp#L97
This func somehow fails

[BlackOfWorld]

If you think just by saying "it doesn't work, fix" will help the developer, then you're wrong. Include console logs and anything you can to help resolve your issue.

[otse]

I wrote a test - it finds my lua files if you change path+L"\\*.lua" to simply path+L"*"

Seems the asterisk is doing something bad

[otse]

Maybe the first result is a directory like .. or .

[otse]

Hi, I fixed this by changing the imgui ScriptItemSelector at
https://github.com/Force67/DawnHook/blob/master/code/dawnhook/Menu.cpp#L21

By swapping the string * vec cast to a wstring

I also .clear() the found_scripts vec.

otse@92382b1#diff-9d1c756acfaa20c8de46590a71f6be80

One bug remains that it skips the first found file. E.g. if you have a, b, c, d.lua, it will skip a.lua

[otse]

I've never used these Windows functions, but I think your loop uses FindNextFile too quickly. A do-while would do better here.

HANDLE hFind = FindFirstFileA(dir.c_str(), &data);      // DIRECTORY

    if (hFind != INVALID_HANDLE_VALUE) {
        do {
            std::cout << data.cFileName << std::endl;
        } while (FindNextFileA(hFind, &data));
        FindClose(hFind);
    }