Uwe Stöhr wrote:
>>> Index: Text.cpp
>>> ===================================================================
>>> --- Text.cpp (revision 39427)
>>> +++ Text.cpp (working copy)
>>> @@ -788,8 +788,11 @@
>>> ++pos;
>>> space_inserted = true;
>>> } else {
>>> - par.insertChar(pos, *cit, font,
>>> bparams.trackChanges);
>>> - ++pos;
>>> + // transfporm tabulators to 4 spaces because
>>> LaTeX cannot handle
>>> + // tabulators and outputting a space instead is
>>> not what we want
>>> + // in FreeSpacing mode
>>> + par.insert(pos, from_ascii(" "), font,
>>> Change(Change::UNCHANGED));
>>
>> why bparams.trackChanges is replaced by Change(Change::UNCHANGED) ?
>
> Because par.insert requires a Change statement, while par.insertChar uses a
> bool (bparams.trackChanges).
> What do you think is correct here?
i dont know CT code. its just suspicious that previous code use variable which
seems to be dependent on the state of CT suddenly changes to some explicit
UNCHAGED state. so i'm asking whether you are sure about it :)
pavel
