<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im"><br><span style="color:rgb(34,34,34)">>    - It requires an English version of Perforce (use the P4_OPTIONS</span><br></div>
<div class="im">>    variable, documented below to pass the -L switch to change the message<br>
>    language if you installation has a different one). It shouldn't be too<br>
> hard to change the regular expressions used for parsing to not key on a<br>
> specific word but on character ranges. I'll look into that if this becomes<br>
> an issue.<br>
<br>
</div>Maybe set P4_OPTIONS automatically to -Lenglish (or whatever) in case it is<br>
not set?<br></blockquote><div><br></div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">I tried that originally but had no effect. Also, on the perforce documentation it says that "Specifies the language to use for error messages from the Perforce service. In order for this flag to work, your administrator must have loaded support for non-English messages in the database", so I didn't feel comfortable adding a flag globably without knowing it might even trigger an error (like invalid language or something like that). </div>
<div style="font-family:'trebuchet ms',sans-serif" class="gmail_default"><br></div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">Perforce offers the P4LANGUAGE environment variable which can be used to configure the client for updates, and we also have the P4_OPTIONS variable, so I think for now it's a good solution/workaround until we have more information.</div>
<div style="font-family:'trebuchet ms',sans-serif" class="gmail_default"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
This will go wrong if the filename contains e.g. "#42". From what your examples<br>
look like this could be changed to " ([a-z]+)$" or if that doesn't file maybe "<br>
([^ ]+)$".<br></blockquote><div><br></div><div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">Good point. I'm changing it to ([^ ]+)$ so it will match </div></div><div><br></div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
+  void DoBodyLine()<br>
+    {<br>
+    if(this->Line[0] == '\t')<br>
<br>
Is it guaranteed that Line is never empty? Otherwise you should test !this-<br>
>Line.empty() before accessing it.<br></blockquote><div><br></div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">Yes, it's guaranteed to be not empty because an empty line would trigger a section change.</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
+  unsigned int i;<br>
+  for(i=0; i<Options.size(); i++)<br>
<br>
This could cause truncation warnings, use size_t better Options::size_type<br>
instead. Or use an iterator.<br></blockquote><div><br></div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">Sounds good, will do that.</div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">
<br></div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">Thanks for your comments.<br><br></div><div style="font-family:'trebuchet ms',sans-serif" class="gmail_default">Pedro</div>
<div style="font-family:'trebuchet ms',sans-serif" class="gmail_default"><br></div></div></div></div>