[Seaside] style: not supported for Canvas' version of (htmllabel)?

Thu Jul 27 06:40:36 UTC 2006

Well I work on a slow, crappy, old VM and have never bitten by this
problem. So my priorities are a bit different. As seen by the code of
Rick, if you send a message to a closed tag, you will get an
exception, even if not the most useful one.

I can only speak for myself but rendering times are already in around
100ms. For me this is already the upper limit. I would not want that
to increase any more, not even in development mode.

I know that having helpful exceptions as early as possible is
important (even it most other Squeak developers disagree because that
would be Java style) but in this case I'd rather not slow down a hot
codepath. Instead I'd document it on the wiki or the FAQ. In general
more (or any) documentation about the Canvas API would be a good
thing. People keep asking the same questions again and again because
there is no documentation and examples use the old renderer.

And someone should inform Bruce Tate ;)

Philippe

2006/7/26, Boris Popov <boris at deepcovelabs.com>:
> Philippe,
>
> Okay, on 3.8 Squeak the same code takes 1136, 919 and 882 milliseconds in 3
> samples, which justifies your desire to avoid doing this as far as I'm
> concerned :) Do you think a deployment test I posted earlier is a good
> compromise?
>
> Cheers!
>
> -Boris
>
> --
> +1.604.689.0322
> DeepCove Labs Ltd.
> 4th floor 595 Howe Street
> Vancouver, Canada V6C 2T5
>
> boris at deepcovelabs.com
>
> CONFIDENTIALITY NOTICE
>
> This email is intended only for the persons named in the message
> header. Unless otherwise indicated, it contains information that is
> private and confidential. If you have received it in error, please
> notify the sender and delete the entire message including any
> attachments.
>
> Thank you.
>
> -----Original Message-----
> From: seaside-bounces at lists.squeakfoundation.org
> [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of Boris Popov
> Sent: Wednesday, July 26, 2006 12:29 PM
> To: The Squeak Enterprise Aubergines Server - general discussion.
> Subject: RE: [Seaside] style: not supported for Canvas' version of
> (htmllabel)?
>
> As far as the impact of doing this, VisualWork's cost is at most around 3
> microseconds per become on a 3GHz P4, which surely adds up over time, but
> not hugely.
>
> a := Object new.
> b := Object new.
> Core.Time millisecondsToRun: [10000 timesRepeat: [a become: b]] --(3
> samples)--> 26 27 25
>
> Cheers!
>
> -Boris
>
> --
> +1.604.689.0322
> DeepCove Labs Ltd.
> 4th floor 595 Howe Street
> Vancouver, Canada V6C 2T5
>
> boris at deepcovelabs.com
>
> CONFIDENTIALITY NOTICE
>
> This email is intended only for the persons named in the message
> header. Unless otherwise indicated, it contains information that is
> private and confidential. If you have received it in error, please
> notify the sender and delete the entire message including any
> attachments.
>
> Thank you.
>
> -----Original Message-----
> From: seaside-bounces at lists.squeakfoundation.org
> [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of Boris Popov
> Sent: Wednesday, July 26, 2006 12:23 PM
> To: The Squeak Enterprise Aubergines Server - general discussion.
> Subject: RE: [Seaside] style: not supported for Canvas' version of
> (htmllabel)?
>
> Specifically,
>
> WATagBrush>>closeTag
>  self document closeTag: self tag.
>  self session application deploymentMode
>    ifFalse: [self become: WAClosedTagBrush new]
>
> New set of VisualWorks changes is attached,
>
> Cheers!
>
> -Boris
>
> --
> +1.604.689.0322
> DeepCove Labs Ltd.
> 4th floor 595 Howe Street
> Vancouver, Canada V6C 2T5
>
> boris at deepcovelabs.com
>
> CONFIDENTIALITY NOTICE
>
> This email is intended only for the persons named in the message
> header. Unless otherwise indicated, it contains information that is
> private and confidential. If you have received it in error, please
> notify the sender and delete the entire message including any
> attachments.
>
> Thank you.
>
> -----Original Message-----
> From: seaside-bounces at lists.squeakfoundation.org
> [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of Boris Popov
> Sent: Wednesday, July 26, 2006 12:16 PM
> To: The Squeak Enterprise Aubergines Server - general discussion.
> Subject: RE: [Seaside] style: not supported for Canvas' version of
> (htmllabel)?
>
> First of all, the cost of doing that in VisualWorks at least is so small,
> you wouldn't even see it, perhaps Squeak is different, I don't know. But
> also, it's easy enough to remove the safety net when #deploymentMode is
> true, isn't it?
>
> Cheers!
>
> -Boris
>
> --
> +1.604.689.0322
> DeepCove Labs Ltd.
> 4th floor 595 Howe Street
> Vancouver, Canada V6C 2T5
>
> boris at deepcovelabs.com
>
> CONFIDENTIALITY NOTICE
>
> This email is intended only for the persons named in the message
> header. Unless otherwise indicated, it contains information that is
> private and confidential. If you have received it in error, please
> notify the sender and delete the entire message including any
> attachments.
>
> Thank you.
>
> -----Original Message-----
> From: seaside-bounces at lists.squeakfoundation.org
> [mailto:seaside-bounces at lists.squeakfoundation.org] On Behalf Of Philippe
> Marschall
> Sent: Wednesday, July 26, 2006 12:10 PM
> To: The Squeak Enterprise Aubergines Server - general discussion.
> Subject: Re: [Seaside] style: not supported for Canvas' version of
> (htmllabel)?
>
> 2006/7/26, Damien Cassou <damien.cassou at laposte.net>:
> > Boris Popov wrote:
> > > Here's an idea, how about we do,
> > >
> > > self become: WAClosedTag new
> > >
> > > when tag gets closed and all further calls to it throw ClosedTag
> exception
> > > from WAClosedTag>>dnu? That may help the new users as well as tired
> seasoned
> > > developers alike ;)
> >
> > Hey,
> >
> > this is something cool. Why don't you put it in squeaksource so that
> > everyone can see the implementation ?
>
> A #become: after each #with:? srsly? Are you aware of the cost of
> this? Afterall this is not ST-72 anymore.
>
> Philippe
> _______________________________________________
> Seaside mailing list
> Seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
>
> _______________________________________________
> Seaside mailing list
> Seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
>
>
>
>