Skip Menu |

This queue is for tickets about the Config-IniFiles CPAN distribution.

Report information
The Basics
Id: 87820
Status: rejected
Priority: 0/
Queue: Config-IniFiles
People
Owner: Nobody in particular
Requestors: keiranmraine [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 2.82
Fixed in: (no value)

History Show all quoted text
  Tue Aug 13 11:39:27 2013 https://www.google.com/accounts/o8/id?id=AItOawnUQUm9oQafaaMf5OlKCWjRF1zV69fbnrA - Ticket created
Subject: Add option to trim leading/trailing space
Hi, It would be a useful addition to the API if there was a configuration option to trim trailing white space (and/or leading). Regards, Keiran M Raine
  Wed Aug 14 09:01:53 2013 SHLOMIF [...] cpan.org - Correspondence added
Hi Keiran, thanks for the wishlist item, but see below for my response. On Tue Aug 13 11:39:27 2013, https://www.google.com/accounts/o8/id?id=AItOawnUQUm9oQafaaMf5OlKCWjRF1zV69fbnrA wrote: Show quoted text
> Hi, > > It would be a useful addition to the API if there was a configuration > option to trim trailing white space (and/or leading).
I'm not too keen to implement it because it will add complexity to the module, may cause files to not be exactly preserved in a round-trip open+save, and may be hard to get the logistics right. However, if you need such a thing, feel free to implement it in a module that wraps Config::IniFiles using a has-a reference. It should not be too hard to do. Do you agree? Regards, -- Shlomi Fish
  Wed Aug 14 09:01:54 2013 The RT System itself - Status changed from 'new' to 'open'
  Sun Aug 20 11:53:04 2017 SHLOMIF [...] cpan.org - Correspondence added
On Wed Aug 14 09:01:53 2013, SHLOMIF wrote: Show quoted text
> Hi Keiran, > > thanks for the wishlist item, but see below for my response. > > On Tue Aug 13 11:39:27 2013, > https://www.google.com/accounts/o8/id?id=AItOawnUQUm9oQafaaMf5OlKCWjRF1zV69fbnrA > wrote:
> > Hi, > > > > It would be a useful addition to the API if there was a configuration > > option to trim trailing white space (and/or leading).
> > I'm not too keen to implement it because it will add complexity to the > module, may cause files to not be exactly preserved in a round-trip > open+save, and may be hard to get the logistics right. However, if you > need such a thing, feel free to implement it in a module that wraps > Config::IniFiles using a has-a reference. It should not be too hard to > do. > > Do you agree? >
Marking this ticket as 'rejected' due to this. Also we waited long enough. Show quoted text
> Regards, > > -- Shlomi Fish
  Sun Aug 20 11:53:05 2017 SHLOMIF [...] cpan.org - Status changed from 'open' to 'rejected'