diced / dlauncher

An application launcher for Linux that is based on Ulauncher
GNU General Public License v3.0
81 stars 6 forks source link

Daemon crash when encountering accented character in desktop files #1

Closed boobin closed 2 years ago

boobin commented 2 years ago

Project built on trunk

log trace:

thread 'main' panicked at 'byte index 2 is not a char boundary; it is inside 'é' (bytes 1..3) of Définition personnalisée pour Firefox Developer Edition', src/fuzzy/mod.rs:69:28

diced commented 2 years ago

This should be fixed in https://github.com/diced/dlauncher/commit/e0d4310bba5c7d9a294b988f517830f6a60c1f30!

dym-dym commented 2 years ago

Sorry to re-open but I still have that issue. here's the crashing output of the daemon : thread 'main' panicked at 'byte index 3 is not a char boundary; it is inside 'é' (bytes 2..4) of Préférences du bureau', library/core/src/str/mod.rs:127:5

diced commented 2 years ago

Did you rebuild dlauncher correctly and are sure that you are using the correct binary?

boobin commented 2 years ago

I juste rebuilt on trunk and the issue seems to be fixed on my system, at least.

dym-dym commented 2 years ago

My bad, I forgot I had the older binary still installed. I thought I removed it before.