To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=66155





------- Additional comments from [EMAIL PROTECTED] Tue Jun  6 10:30:51 -0700 
2006 -------
Ause - the old nitpicker - found a small glitch in one of the REs (we spend 20
min to understand it :-) ... and I found another one :-)

First (the line with the first + sign in the patch): The square backets in the
RE are exchanged for round ones. This isn't right, is it? This either doesn't
match (most of the cases) or match in funny ways. See attached snippet.

Secondly and here we are nitpicking :-): 
Please exchange the mentioned round brackets to square brackets in the "if" line
of the snippet . Now, if there is a whitespace before the quote sign on both
ends of the value string ie,
foo= "bar " 
then the RE which kills the surrounding quotes of the value will also kill the
last space inside the quote which still belongs to the value.

my $str = 'BAR1= "foo "';
print "$str\n";
if ( $str =~ m/(\w(\w\d)+)=(.*)$/ ) {
    my $var = $1;
    my $val = $2;
    print "$var\n";
    print "$val\n";
    $val =~ s/(\s*['"])?(.*)\1\s*$/\2/;
    print "x${val}x\n";
}

I told you we are nitpicking ... :-)

---------------------------------------------------------------------
Please do not reply to this automatically generated notification from
Issue Tracker. Please log onto the website and enter your comments.
http://qa.openoffice.org/issue_handling/project_issues.html#notification

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to