Jason Gibson
Jason Gibson
The change in error messages appears to have originated in ce32549bc6f2690b0993ab0c3e4c66aa37c8c86d. The use of lua_error() vs luaL_error() is the key in type_panic_string() and type_panic_c_str(). Issue 906 also references a similar...
Here is a diff (off of v3.3.0) that fixes it for my use-cases. I suspect that anywhere that push_type_panic_string() and lua_error() are used together might need changing to luaL_error(), though....
@shockdot Did you try the [patch](https://github.com/ThePhD/sol2/issues/1386#issuecomment-1236034531)? Does it improve the error messages in your tests? Perhaps if you wanted to repackage the raw diffs as a pull request, it would...
@shockdot Ah, knowing what file to apply the patch to might help :) The diff was against the single-header version of the 3.3.0 release - I don't know which originating...
Apologies for the delay in getting some files for this. Here's a repro (with different files than the initial message in this report) using 'Difftastic 0.56.1 (d9d6401 2024-03-05, built with...