On 17 Aug 2014, at 08:08, Marcus Denker <marcus.denker@inria.fr> wrote:
On 16 Aug 2014, at 17:21, Ben Coman <btc@openInWorld.com> wrote:
Gareth Cox wrote:
AST-Core changes seem to have broken Seaside REST. I've sent messages to Seaside lists and various people but haven't got a response. Pharo3, Seaside3.1 on Mac OS X. -- Gareth Cox IT Manager/Developer Inspired Org (PTY) Ltd email: gareth@inspired.org tell: +27 (0)21 531 5404 cell: +27 (0)78 374 9035
I'm not familiar with either AST or Seaside, but a quick search found these on the issue trackers. Are they related? https://code.google.com/p/seaside/issues/detail?id=807 https://pharo.fogbugz.com/default.asp?13246
The issue 13246 is that, yes.
The implementation of #argumentNamesOf: course low-level details, in that case the AST.
Now we changed the AST in Pharo3 to be the RB AST which provides a lot of compatibility methods, but not the one needed here. (which makes no sense for the new AST)
In the meantime added #argumentNames to CompiledMethod (because many clients need it⦠why should everyone do their own version?), so in Pharo3 you can just write that method as:
argumentNamesOf: aCompiledMethod ^ aCompiledMethod argumentNames
The case https://code.google.com/p/seaside/issues/detail?id=807 needs to be re-opened. Marcus