Ticket #3586 (closed defect: fixed)

Opened 9 years ago

Last modified 9 years ago

[PATCH] mcedit: fix "Lower case selection" command in User Menu

Reported by: rimf Owned by: zaytsev
Priority: trivial Milestone: 4.8.16
Component: mcedit Version: master
Keywords: lower case Cc:
Blocked By: Blocking:
Branch state: no branch Votes for changeset:

Description

"Lower case selection" command in User Menu works as "Upper case selection" command (copy-paste bug).

Attachments

mcedit-fix-lower-case-selection-command.patch​ (511 bytes) - added by rimf 9 years ago.
mc-3586-fix-wrong-lowercase-sed-syntax.patch​ (1.1 KB) - added by and 9 years ago.

Change History

Changed 9 years ago by rimf

comment:1 Changed 9 years ago by and

Thanks,
nice finding.

lets strip cat and pipe too

sed 's/\(.*\)/\L\1/' $TMPFILE >%b

comment:2 Changed 9 years ago by zaytsev-work

Hi Andreas, could you please make a patch for both then? Thanks!

Changed 9 years ago by and

comment:3 Changed 9 years ago by zaytsev

  • Blocked By 3547 added

comment:4 Changed 9 years ago by zaytsev

  • Owner set to zaytsev
  • Status changed from new to accepted
  • Milestone changed from Future Releases to 4.8.16

Committed into the cleanup branch, thank you Andreas!

comment:5 Changed 9 years ago by andrew_b

  • Blocked By 3547 removed

comment:6 Changed 9 years ago by andrew_b

  • Status changed from accepted to testing
  • Resolution set to fixed

Applied as [7da330ce770082416bea757574ed333ccd163553].

comment:7 Changed 9 years ago by andrew_b

  • Status changed from testing to closed
Note: See TracTickets for help on using tickets.