s3rename icon indicating copy to clipboard operation
s3rename copied to clipboard

Allow use of translate sed command "y/" and "tr/"

Open jamesmcm opened this issue 5 years ago • 3 comments

Currently there is no support for the y/ translate command i.e. y/ABCDEFGHIJKLMNOPQRSTUVWXYZ/abcdefghijklmnopqrstuvwxyz/ for translating upper-case text to lower-case.

This should first be patched in the sedregex crate, see this issue.

If that is not possible, this could be implemented separately (i.e. its own function).

jamesmcm avatar May 03 '20 14:05 jamesmcm

Hi @jamesmcm, would love to work on this issue.

developerfred avatar May 03 '20 21:05 developerfred

@developerfred excellent, I think it'd be best to implement it directly in the sedregex crate if possible.

And it's probably more efficient to just implement it as its own function (rather than building separate regexes).

jamesmcm avatar May 04 '20 07:05 jamesmcm

@jamesmcm thanks, I will fork this project on gitlab.

developerfred avatar May 05 '20 17:05 developerfred