[Synedit-user] Another Bug - SynTextDrawer

[=?US-ASCII?Q?Juh=E1sz L. <lj...@al...>]

I've used the version 2.0.4 for a couple of days. After changing code dealing with the user interface (nothing to do a synedit component) when the application tries to destroy a synedit component the following assert fails:


{$IFDEF HE_ASSERT}
    ASSERT(LockCount < RefCount,
      'Call DeactivateFontsInfo before calling this.');
{$ELSE}
    ASSERT(LockCount < RefCount);
{$ENDIF}

(lines 485 - 490). My solution was to place the assert into a comment.

-----Original message-----
From: "Alan Garny" ag...@he...
Date: Mon, 26 Feb 2007 15:39:50 +0100
To: syn...@li...
Subject: [Synedit-user] Potential problems in SynEdit.pas (lines 3332/3333
	and 3338/3339)

>  Hi, I have just upgraded to SynEdit 2.0.4 (thanks for the great component by
>  the way!) and believe to have come across a couple of bugs. They are located
>  at lines 3332/3333 and 3338/3339. They respectively read:
>  
>                AddHighlightToken(sToken, nTokenPos, nTokenLen,
>  attr.Foreground,
>                  attr.Background, []);
>  
>  and
>  
>                AddHighlightToken( SynLineBreakGlyph, nTokenPos + nTokenLen,
>  1,
>                  attr.Foreground, attr.Background, []);
>  
>  and the problem occurs when 'attr' is not assigned, which unfortunately
>  happened to one of my users (argh!). There is an easy workaround (after
>  looking at other uses of 'attr'), which results in the above being replaced
>  with:
>  
>                if Assigned(attr) then
>                  AddHighlightToken(sToken, nTokenPos, nTokenLen,
>  attr.Foreground,
>                    attr.Background, [])
>                else
>                  AddHighlightToken(sToken, nTokenPos, nTokenLen, colFG,
>  colBG,
>                    []);
>  
>  and
>  
>                if Assigned(attr) then
>                  AddHighlightToken( SynLineBreakGlyph, nTokenPos + nTokenLen,
>  1,
>                    attr.Foreground, attr.Background, [])
>                else
>                  AddHighlightToken( SynLineBreakGlyph, nTokenPos + nTokenLen,
>  1,
>                    colFG, colBG, []);
>  
>  respectively. Now, the question to you guys is whether the above is indeed
>  what should have been written in the first place? I would indeed make sure
>  that the new version of my software contains the right code...
>  
>  	Cheers, Alan.
>  
>  
>  -------------------------------------------------------------------------
>  Take Surveys. Earn Cash. Influence the Future of IT
>  Join SourceForge.net's Techsay panel and you'll get the chance to share your
>  opinions on IT & business topics through brief surveys-and earn cash
>  http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
>  _______________________________________________
>  Synedit-user mailing list
>  Syn...@li...
>  https://lists.sourceforge.net/lists/listinfo/synedit-user




_____________________

Téléchargez P3dO : visualisez et organisez vos objets 3D, vos images, vos photos etc...
P3dO gère et visionne plus de 40 formats 2D et près de 20 formats 3D.

http://www.alterlinks.fr/p3do/