Ticket #2544 (closed defect: fixed)

Opened 8 years ago

Last modified 7 years ago

search string with leading space

Reported by: victor-homyakov Owned by: slavazanko
Priority: major Milestone: 4.8.0-pre1
Component: mc-config-ini Version: 4.7.5
Keywords: Cc: gotar@…
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master committed-stable

Description

What version of Midnight Commander is used?

$ mc -V
GNU Midnight Commander, версия 4.7.5
Скомпилирован с библиотекой GLib версии 2.20.1
С библиотекой S-Lang и с базой данных terminfo
Со встроенным редактором
C поддержкой внутренней командной оболочки
С поддержкой фоновых операций
С поддержкой мыши в xterm
С поддержкой событий X11
С поддержкой интернационализации
С поддержкой многих кодировок
Виртуальная файловая система: cpiofs, tarfs, sfs, extfs, ftpfs, fish, smbfs
Тип данных: char: 8; int: 32; long: 32; void *: 32; size_t: 32; off_t: 64;

What steps will reproduce the problem?

  1. View some text (log) file (F3), or edit (F4).
  2. Open search dialog (F7).
  3. Enter string starting with space and ending with space (I used " ERROR " without quotes), press Enter
  4. Press F7 to open search dialog again.

What is the expected output?
Original string in search dialog (" ERROR " without quotes)

What do you see instead?
"ERROR " (without leading space!)

Change History

comment:1 Changed 8 years ago by andrew_b

  • Component changed from mc-search to mc-config-ini
  • Milestone changed from 4.8 to 4.8.0-pre1

Leading spaces are stripped when config is read from file.

comment:2 Changed 8 years ago by slavazanko

  • Status changed from new to accepted
  • Owner set to slavazanko

comment:3 Changed 8 years ago by slavazanko

  • severity changed from no branch to on review

created branch 2544_search_trim_leading_spaces
Initial changeset:c43462479d7736b6fabeae0d93909f1bba5ebcb3

Review, please.

comment:4 Changed 8 years ago by angel_il

  • Votes for changeset set to angel_il

comment:5 Changed 8 years ago by slavazanko

  • Votes for changeset angel_il deleted
  • severity changed from on review to on rework

Sorry, have troubles with double encoding while save/load history

comment:6 Changed 8 years ago by slavazanko

  • severity changed from on rework to on review

New initial changeset:3d0ab70af5d36d8391505d0fdefea5106da9e616
Review again, please.

comment:7 Changed 8 years ago by andrew_b

  • Votes for changeset set to andrew_b

comment:8 Changed 8 years ago by angel_il

  • Votes for changeset changed from andrew_b to andrew_b angel_il
  • severity changed from on review to approved

comment:9 Changed 8 years ago by slavazanko

  • Keywords stable-candidate added
  • Status changed from accepted to testing
  • Votes for changeset changed from andrew_b angel_il to committed-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:10 Changed 8 years ago by slavazanko

  • Status changed from testing to closed
  • Keywords stable-candidate removed
  • Votes for changeset changed from committed-master to committed-master committed-stable

comment:11 Changed 8 years ago by gotar

  • Cc gotar@… added
  • Branch state set to no branch

comment:12 Changed 7 years ago by andrew_b

  • Branch state changed from no branch to merged
Note: See TracTickets for help on using tickets.