The attached removes a few pointless calls from QInclude.cpp that do
nothing but cause a warning to be written to the console. OK to commit?
rh
--
==
Richard G Heck, Jr
Professor of Philosophy
Brown University
On Thursday 24 May 2007 07:28:40 Richard Heck wrote:
The attached removes a few pointless calls from QInclude.cpp that do
nothing but cause a warning to be written to the console. OK to commit?
The purpose of the code seems to be resetting the options, if you are sure
that it works without
José Matos wrote:
On Thursday 24 May 2007 07:28:40 Richard Heck wrote:
The attached removes a few pointless calls from QInclude.cpp that do
nothing but cause a warning to be written to the console. OK to commit?
The purpose of the code seems to be resetting the options, if you are
Hmm, I see. Let's ask Bo about this, as I think he must have added these
calls.
I haven't been able to cause a problem after deleting these lines. The
reason is that the InsetInclude::latex() routine just ignores the
options, anyway, if we're not dealing with a listings include. The
advantage
Bo Peng wrote:
I haven't been able to cause a problem after deleting these lines. The
reason is that the InsetInclude::latex() routine just ignores the
options, anyway, if we're not dealing with a listings include. The
advantage to deleting them turns out to be that if you switch from
The attached removes a few pointless calls from QInclude.cpp that do
nothing but cause a warning to be written to the console. OK to commit?
rh
--
==
Richard G Heck, Jr
Professor of Philosophy
Brown University
On Thursday 24 May 2007 07:28:40 Richard Heck wrote:
> The attached removes a few pointless calls from QInclude.cpp that do
> nothing but cause a warning to be written to the console. OK to commit?
The purpose of the code seems to be resetting the options, if you are sure
that it works without
José Matos wrote:
> On Thursday 24 May 2007 07:28:40 Richard Heck wrote:
>
>> The attached removes a few pointless calls from QInclude.cpp that do
>> nothing but cause a warning to be written to the console. OK to commit?
>>
> The purpose of the code seems to be resetting the options, if
Hmm, I see. Let's ask Bo about this, as I think he must have added these
calls.
I haven't been able to cause a problem after deleting these lines. The
reason is that the InsetInclude::latex() routine just ignores the
options, anyway, if we're not dealing with a listings include. The
advantage
Bo Peng wrote:
>> I haven't been able to cause a problem after deleting these lines. The
>> reason is that the InsetInclude::latex() routine just ignores the
>> options, anyway, if we're not dealing with a listings include. The
>> advantage to deleting them turns out to be that if you switch from
10 matches
Mail list logo