Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Broken case-sensitive search in editor/viewer/diffviewer #3069

Closed
mc-butler opened this issue Sep 6, 2013 · 6 comments
Closed

Broken case-sensitive search in editor/viewer/diffviewer #3069

mc-butler opened this issue Sep 6, 2013 · 6 comments
Assignees
@aborodin
Labels
area: search Search subsystem prio: medium Has the potential to affect progress ver: 4.8.9 Reproducible in version 4.8.9
Milestone
4.8.11

Comments

@mc-butler
Copy link

Important

This issue was migrated from Trac:

Origin https://midnight-commander.org/ticket/3069
Reporter andrew_b (@aborodin)

There are some problems in search was raised after #3026.

Example.
Locale is KOI8-R, file encoding is UTF-8. Note: those encodings are not same.
File content is following (in Russian):

йцукен
Йцукен

The difference is in first line char only: й (lowercase letter) and Й (uppercase letter).
The search of й gives the result й independently of case sensitivity. Й isn't found.
If switch "All charsets" on, nothing is found.
@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 6, 2013 at 9:02 UTC (comment 1)

  • Summary changed from Broken case-insensitive search in editor/viewer/diffviewer to Broken case-sensitive search in editor/viewer/diffviewer

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 6, 2013 at 9:36 UTC (comment 2)

  • Branch state changed from no branch to on review
  • Milestone changed from Future Releases to 4.8.11
  • Owner set to andrew_b
  • Status changed from new to accepted

Branch: 3069_case_sens_search_fix
Initial [aede1552a20602214e6959a52192d3f22e44e1a2]

@mc-butler
Copy link
Author

Changed by slavazanko (@slavaz) on Sep 9, 2013 at 12:55 UTC (comment 3)

  • Votes set to slavazanko

@mc-butler
Copy link
Author

Changed by angel_il (@ilia-maslakov) on Sep 20, 2013 at 7:54 UTC (comment 4)

  • Branch state changed from on review to approved
  • Votes changed from slavazanko to slavazanko angel_il

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 21, 2013 at 9:54 UTC (comment 5)

  • Branch state changed from approved to merged
  • Resolution set to fixed
  • Status changed from accepted to testing
  • Votes changed from slavazanko angel_il to committed-master

Merged to master: [852b885].

git log --pretty=oneline cc980f3..852b885

@mc-butler
Copy link
Author

Changed by andrew_b (@aborodin) on Sep 21, 2013 at 9:56 UTC (comment 6)

  • Status changed from testing to closed

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

@aborodin aborodin

Labels
area: search Search subsystem prio: medium Has the potential to affect progress ver: 4.8.9 Reproducible in version 4.8.9
Milestone
4.8.11
Development

No branches or pull requests
2 participants
@aborodin @mc-butler