Richard Heck <rgh...@lyx.org> writes:

| On 09/17/2012 03:19 AM, Juergen Spitzmueller wrote:
>> The branch, master, has been updated.
>>
>> - Log -----------------------------------------------------------------
>>
>> commit e8a01d099a7ecbe5059cbdf0aa0aab16e9862cf6
>> Author: Juergen Spitzmueller <sp...@lyx.org>
>> Date:   Mon Sep 17 08:59:41 2012 +0200
>>
>>      Calibrate log file parser
>>           Filenames embraced in <...> can occur anywhere on the line
>> and multiple times. This fixes for me the case that graphics
>> included via ERT were not tracked. It probably also fixes #8336.
>>           This is a candidate for branch.
>>
| Go ahead and put this in, Jurgen. But we should e on the lookout for
| any issues. I'm planning to start the release process fairly soon
| (string freeze, etc).
>
| Richard
>
>> diff --git a/src/LaTeX.cpp b/src/LaTeX.cpp
>> index 110d21c..ecd255f 100644
>> --- a/src/LaTeX.cpp
>> +++ b/src/LaTeX.cpp
>> @@ -995,6 +995,68 @@ bool completeFilename(string const & ff, DepTable & 
>> head)
>>      return handleFoundFile(ff, head);
>>   }
>>   +
>> +int iterateLine(string const token, regex const reg, string const closing,
>> +            int fragment_pos, DepTable & head)

'closing' - why not const ref? Are there contex issues here, that makes
creating a copy required?

-- 
   Lgb

Reply via email to