Juergen Vigna <[EMAIL PROTECTED]> writes:
| On 29-Apr-2002 Dekel Tsur wrote:
>> On Mon, Apr 29, 2002 at 06:01:27PM +0200, Lars Gullik Bjønnes wrote:
>>> Why do you say that 1.2.0cvs is not able to read files with the latex
>>> layout? Hmm it seems that we test for a valid layout a bit early in
>>> buffer.C... so that the compability reading does not kick in before we
>>> have already changed to using defaultLayoutName...
>>>
>>> Do anyone have a test file for me? That is f.ex two plain paragraphs
>>> and a latex paragraph in between?
>>
>> Attached.
>
| Well it seems to me this is perfectly read!
>
| Anyway to anser Lars
>
>>I'd like some one to test this before it is applied.
>> What are a relevant test file?
>
| As John told you the test file is on LyXBugs (but you seem to have problems
| with it as you told me). Anyway I explain you what is wrong. We set a font
| with the default language (I have as default "english"). So we have this font
| and it's language is "english". Now we read the \\language tag and set the
| default language to f.ex. "norsk" the font is NOT set to this language.
>
| This is only done later when reading the \\layout tag and there the check
| for "latex" is too early, before we change the language in the font with
>
>> font = LyXFont(LyXFont::ALL_INHERIT, params.language);
>
| So an alternative fix would be move the block:
>
| #ifndef NO_COMPABILITY
| if (compare_no_case(layoutname, "latex") == 0) {
| ert_comp.active = true;
| ert_comp.font = font;
| }
| #endif
>
| To after the settings of the the above fonttag! No to tell the truth now
| that I think about it we should just move the above font asignment, BEFORE
| the check for "latex", this would then fix also the reading of caption when
| we activate that one ;)
>
| Here is the new patch for fixing this bug.
And this is a different bug from the one I answered to, right?
Anyway I think your patch looks ok.
--
Lgb
