Skip to content

Conversation

Neloreck
Copy link
Contributor

@Neloreck Neloreck commented Feb 6, 2025

Similar to #26, without log update
Over time I have noticed that it is an actual issue + LuaJIT adds '@' as prefix for file paths (

if (source.StartsWith("@"))
)

What

  • Add case insensitive strings comparison when resolving LUA sources, fixes problems when names or folders are in camel case
  • As result, I am able to set breakpoints for files with upper case in pathname/filename

How

  • Generic comparison changes

Notes

  • I assume it is still as problematic solution as we already have. Windows OS does not care about case, but unix based operating systems (if extension is somehow used with them?) require strict equality of path.
  • Root of the problem may be originally in place where paths are transformed to lowercase before storing them

Tested

  • Tested in VS community 2022

@Xottab-DUTY
Copy link

Normalized log format to be more consistent and readable with similar lines

This wasn't ported from #26. I guess, it needs to be ported or should be removed from the description of this PR :))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants