[Seaside-dev] Context Initialization was: Fwd: [Seaside] Seaside and Exupery patch

[Philippe Marschall]

Hi

Bryce Kampjes is working on Seaside on Exupery and suggested the
following changes:

Currently we have

Continuation >> #initializeFromContext: aContext
	| valueStream context |
	valueStream := WriteStream on: (Array new: 20).
	context := aContext.
	[context notNil] whileTrue:
		[valueStream nextPut: context.
		1 to: context class instSize do: [:i | valueStream nextPut: (context
instVarAt: i)].
		1 to: context localSize do: [:i | valueStream nextPut: (context localAt: i)].
		context := context sender].
	values := valueStream contents

With this change we would have

Continuation >> #initializeFromContext: aContext
	| valueStream context |
	valueStream := WriteStream on: (Array new: 20).
	context := aContext.
	[context notNil] whileTrue:
		[context serialiseOn: valueStream.
		context := context sender].
	values := valueStream contents

and

ContextPart >> #serialiseOn: aStream
	aStream nextPut: self.
		1 to: self class instSize do: [:i | aStream nextPut: (self instVarAt: i)].
		1 to: self localSize do: [:i | aStream nextPut: (self localAt: i)].

Does anybody see any problems with this? Would ContextPart >>
#serialiseOn: better be moved to Squeak*?

Cheers
Philippe

---------- Forwarded message ----------
From: bryce at kampjes.demon.co.uk <bryce at kampjes.demon.co.uk>
Date: 03.12.2007 21:47
Subject: [Seaside] Seaside and Exupery patch
To: seaside at lists.squeakfoundation.org
Cc: exupery at lists.squeakfoundation.org



I've placed Seaside2.8a1-wbk.523 on to the Seaside SqueakSource
repository. This is a patch that moves the serialisation of
contexts from Continuation into ContextPart. This is to allow
Seaside to work with Exupery.

Is it possible to get this merged into the development branch of
Seaside?

Yes, the latest development versions of Exupery will run Seaside, no,
it's not yet practical. Expect bugs and don't expect a performance
improvement yet.

Bryce
_______________________________________________
seaside mailing list
seaside at lists.squeakfoundation.org
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside