Sometimes we'll get here with urls that are not normalized
This leads to crash
Details
Details
Diff Detail
Diff Detail
- Repository
- R33 KDevPlatform
- Branch
- 5.0
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
this is not the correct place to handle this. the callers must be adapted. Yes, this is more work. But it also means it can be done much much faster.
Comment Actions
Btw, using Path instead of QUrl is often all you need to "fix" this - the code will be faster as well.
Comment Actions
Oh well, should have kept the backtrace. Are you suggesting to introduce a ::chainForDocument(Path) method?
Comment Actions
Maybe. Or simply use the one that takes an IndexedString. I think what you want is to use Path::toIndexed wherever you currently use QUrl. QUrl is simply too easy to get wrong