RE: [XaraXtreme-dev] File => Close crashes

[Alex Bligh <alex@xxxxxxxxxxx>]

But what was the original purpose of the IsScreenCamView check? I had
presumed distinguishing it from (say) a print view.

Alex

--On 05 April 2006 17:23 +0100 Gerry Iles <GerryI@xxxxxxxx> wrote:

> Yes, there used to be a class hierarchy:
>
> ScreenCamView
> ScreenView
> CCamView
> wxView/CView
>
> This was deemed pretty pointless so the top two classes were merged into
> CCamView when I reorganised things a while back.  That test should now
> be totally redundant.
>
> I'm pretty certain that the selected document and view should be
> cleared, resulting in the various SELCHANGED messages being broadcast
> before the document and view get deleted and that this isn't happening
> for some reason.
>
> There should definitely never be any SELCHANGED type messages sent
> around where the old pointer points at something that has been deleted
> either notionally, or actually with delete.
>
> The fact that you can't seem to have more than one view visible at a
> time in the linux version hides a lot of the complexity of the view
> activation code e.g. when you switch view, the tools render the blobs
> off the old view but that is now behind the new view and doesn't
> actually redraw anything.
>
> Gerry
>
> -----Original Message-----
> From: owner-dev@xxxxxxxxxxxxxxxx [mailto:owner-dev@xxxxxxxxxxxxxxxx] On
> Behalf Of Alex Bligh
> Sent: 05 April 2006 17:02
> To: dev@xxxxxxxxxxxxxx
> Cc: Alex Bligh
> Subject: RE: [XaraXtreme-dev] File => Close crashes
>
>
>
> --On 05 April 2006 16:57 +0100 Gerry Iles <GerryI@xxxxxxxx> wrote:
>
> > I don't think it has anything specifically to do with the call to
> > IsScreenCamView.
>
> IsScreenCamView doesn't exist any more for some reason (view class
> rearrangement I presume). I actually commented it out in more than
> one tool conversion, so it might be worth a quick grep for it.
>
> Alex