Closed
Bug 395734
Opened 18 years ago
Closed 18 years ago
initial activescript prototype
Categories
(Tamarin Graveyard :: Virtual Machine, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: markh, Assigned: markh)
Details
Attachments
(1 file)
79.69 KB,
patch
|
Details | Diff | Splinter Review |
I'm attaching an initial implementation of an ActiveScripting engine. Note it does not work yet, but I'm hoping to check it in so I can ask sensible questions regarding interfacing with the VM. See README.txt in the patch for details on what does work and how to use it.
Note that this engine is still officially a "prototype" - a final implementation may take a different approach.
The patch itself creates a tamarin-central/axscript directory. The project itself (ie, project files, main .cpp filename, class names, etc) is generally named 'axtam'. (I used 'axscript' as the directory name instead of 'axtam' as I guessed the former would be self-evident more than the latter). All files except README.txt and the VS project files are stored with \n line endings, although this is not obvious from the patch.
I'm not (yet) asking for a formal review of the code (although that would be welcome). I'm primarily checking the 'axscript' and 'axtam' names are acceptable, especially given this is a 'prototype' and may not resemble the final implementation, and that the location of the axscript directory (as a sibling of 'esc', 'shell' etc) is acceptable.
Assignee | ||
Comment 1•18 years ago
|
||
Comment 2•18 years ago
|
||
Why are you putting this engine in the Tamarin tree? It seems to me that this code embeds Tamarin and doesn't need to live in the same repository.
Comment 3•18 years ago
|
||
Benjamin, we talked about this a bit on last week's conference call. Nobody had strong feelings either way. If you think of ScreamingMonkey as an installer for Tamarin ;) then it definitely belongs in tamarin-central. Mostly we didn't see much point separating them.
Comment 4•18 years ago
|
||
At today's weekly tamarin-devel meeting, there was more discussion about source location, and we still favor keeping this in tamarin-central:
* tamarin-central is still small, this isn't bloating it.
* We gain some (> 0) benefit in usability and discoverability, aiding quality.
* We can get more prompt build and automated test error feedback.
* We (I do, at any rate) hope to see this ship on Windows with Tamarin.
We can certainly revisit. The trade-offs may change. But for now this looks like the right decision to those who spoke up and have a stake.
/be
Assignee | ||
Comment 5•18 years ago
|
||
checked in
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•