Re: genini patch to support the new message field

2010-03-01 Thread Brendan Conoboy
On 02/28/2010 04:51 PM, Christopher Faylor wrote: I think it makes more sense to test for message initially and, if found skip over the keyword and parse the rest of the line as normal: Does this work? Yes- genini produces the same output with either of our two patches. Are there any other ne

Re: genini patch to support the new message field

2010-02-28 Thread Christopher Faylor
On Fri, Feb 26, 2010 at 09:03:46PM -0700, Brendan Conoboy wrote: >2010-02-26 Brendan Conoboy > > * genini (get): Handle special 'message' format case. > >*** >*** 84,92 > my $key = shift; > my $val = shift; > >! if (substr($val, 0, 1) ne '"') { >! $v

genini patch to support the new message field

2010-02-26 Thread Brendan Conoboy
A recent addition of a message field to Cygwin's setup.hint broke genini. The introduction of a new unquoted token between the key and the value confused the get subroutine. The attached patch adds a special case for message fields, allowing correct parsing in this particular case. I'm not