<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri","sans-serif";}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">Brian,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">You committed a fix to vtkDelimitedTextReader on 5/31/2011 (see below). There's still a problem with trailing whitespace. Whitespace between the end of the string delimiter and the next field delimiter is not discarded.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Currently the line:<o:p></o:p></p>
<p class="MsoNormal">'1, "word" ,2'<o:p></o:p></p>
<p class="MsoNormal">is parsing to:<o:p></o:p></p>
<p class="MsoNormal">"1";"word ";"2" <-- trailing whitespace around "word" is unexpected.<o:p></o:p></p>
<p class="MsoNormal">I would expect:<o:p></o:p></p>
<p class="MsoNormal">"1";"word";"2" <-- no extraneous whitespace around "word". This is what I want.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Is there any chance you'd be willing to make this change? I'd make this change myself, but it's not clear to me what impact this may have on clients of the object. Note that I've recently submitted the following enhancements to vtkDelimitedTextReader:
http://review.source.kitware.com/3361.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks!<o:p></o:p></p>
<p class="MsoNormal">Scott<b><o:p></o:p></b></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">===========================================<o:p></o:p></p>
<p class="MsoNormal">commit 7b5c9ae55717d3485ec7cc7f3e5380221c8bc1d2<o:p></o:p></p>
<p class="MsoNormal">Author: Brian Helba <brian.helba@kitware.com><o:p></o:p></p>
<p class="MsoNormal">Date: Tue May 31 15:32:08 2011 -0400<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> BUG: String parsing in vtkDelimitedTextReader<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> When using string delimiters, any content (usually whitespace)<o:p></o:p></p>
<p class="MsoNormal"> between the start of a field and the start of a string delimiter<o:p></o:p></p>
<p class="MsoNormal"> should be discarded.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> Previously, the line:<o:p></o:p></p>
<p class="MsoNormal"> '1, "word",2'<o:p></o:p></p>
<p class="MsoNormal"> was parsing to:<o:p></o:p></p>
<p class="MsoNormal"> "1"; " word"; "2"<o:p></o:p></p>
<p class="MsoNormal"> it will now parse to:<o:p></o:p></p>
<p class="MsoNormal"> "1"; "word"; "2"<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> Change-Id: I4542bbe8de9466bbdda0ac37214b85b99bf68724<o:p></o:p></p>
<p class="MsoNormal">=============================================<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><b><span style="font-size:14.0pt;font-family:"Arial","sans-serif";color:#005293">Scott Fowler</span></b><b><span style="font-size:12.0pt;font-family:"Arial","sans-serif";color:navy"><br>
</span></b><span style="font-size:9.0pt;font-family:"Arial","sans-serif";color:#707070">Software Development Engineer
<br>
Phone: 425.460.8265<br>
Email: </span><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#4F81BD"><a href="mailto:s.fowler@tecplot.com"><span style="color:#4F81BD">s.fowler@tecplot.com</span></a></span><span style="font-family:"Arial","sans-serif""><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-family:"Arial","sans-serif""> <o:p></o:p></span></p>
<p class="MsoNormal"><b><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:#005293">Tecplot Inc. | Master the View</span></b><b><span style="font-size:10.0pt;font-family:"Arial","sans-serif";color:navy"><br>
</span></b><span style="font-size:9.0pt;font-family:"Arial","sans-serif";color:#707070">3535 Factoria Blvd. SE, Ste. 550<br>
Bellevue, WA USA 98006<br>
Fax: 425.653.9200<br>
Web:</span><span style="font-size:9.0pt;font-family:"Arial","sans-serif";color:gray">
</span><span style="color:#4F81BD"><a href="http://www.tecplot.com/" title="http://www.tecplot.com/"><span style="font-size:9.0pt;font-family:"Arial","sans-serif";color:#4F81BD">www.tecplot.com</span></a></span><o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<br>
<hr>
<font face="Arial" color="Gray" size="1">This e-mail and any files transmitted with it are confidential and are intended solely for the use of the individual or entity to whom they are addressed. If you are NOT the intended recipient or the person responsible
for delivering the e-mail to the intended recipient, be advised that you have received this e-mail in error and that any use, dissemination, forwarding, printing or copying this e-mail is strictly prohibited.<br>
</font>
</body>
</html>