[Date Prev][Date Next][Thread Prev][Thread Next][Thread Index]

RE: [XaraXtreme-dev] RE: [XaraXtreme-bugzilla] [Bug 1125] Bevel tool does not work properly on Text



Well, it might have been but the change makes no sense conceptually.  A
TextChar object is not a compound node and never can be so any change
that makes it so is fundamentally wrong...

I think it was either accidental (e.g. the function was accidentally
moved from the TextStory class) or not very well considered due to
unfamiliarity with the code...

Gerry

-----Original Message-----
From: owner-dev@xxxxxxxxxxxxxxxx [mailto:owner-dev@xxxxxxxxxxxxxxxx] On
Behalf Of Alex Bligh
Sent: 04 July 2006 18:49
To: dev@xxxxxxxxxxxxxx
Cc: Alex Bligh
Subject: Re: [XaraXtreme-dev] RE: [XaraXtreme-bugzilla] [Bug 1125] Bevel
tool does not work properly on Text

Gerry Iles wrote:
> Martin,
> 
> In fact, it appears that this override of IsCompound was present in
r714
> which was before any of your text handling stuff was added.
> 
> Anyone remember the error "LocaliseCommonAttributes called on
> non-Compound Node" when doing things with text stories early on in the
> port?

I have better archaeology. It was thus on rev 385 of the very first
svn repository. It must have been a very early change indeed. Which
is most peculiar as r385 was nowhere near rendering text.

Are you sure it was not at some stage converted to do that in
SourceSafe, someone took the patch over, then SourceSafe got
reverted but no-one carried the patch through to svn?

Alex