Search
Monday, 15th of October 2018, 8:47:57 UTC
8:52:57
beach
In the SICL MOP, I define this additional function (not specified by the AMOP) named ENSURE-METHOD.
8:53:16
beach
That's the one called by the expansion of DEFMETHOD.
8:54:07
beach
And that's the one that calls ADD-METHOD.
8:54:45
no-defun-allowed
i thought the AMOP did have ENSURE-METHOD
8:54:51
no-defun-allowed
maybe that was ENSURE-GENERIC-FUNCTION or something
8:54:59
beach
Yes, and ENSURE-CLASS.
8:55:04
beach
But not ENSURE-METHOD.
8:55:18
no-defun-allowed
amop does mention ensure-method
8:55:21
beach
So the definition of ENSURE-METHOD needs to move one step down. The one currently in E2 should move to E1, and the one currently in E3 should move to E2.
8:55:42
no-defun-allowed
it's on page 38 of my copy
8:55:54
no-defun-allowed
oh yes, that would probably be the spec part too
8:56:05
beach
Let me go find my copy...
8:56:34
heisig
It is part of Chapter 1 and of the Closette implementation in Appendix D.
8:57:17
beach
Ah, OK. So that must be where I got it from.
8:57:26
beach
Still, it is not in the specification.
9:01:06
beach
OK, I think I have a plan of action.
9:01:48
beach
The ENSURE-METHOD that is currently loaded into E2, should be loaded into E1 instead.
9:02:20
beach
I then need to move the definition of it to E1, which is a simple operation.
9:03:39
beach
The only problem with that idea is that, in the next phase, I need to load it into E2 and move the definition into E3, but then I will clobber the definition in E2.
9:04:20
beach
But that should not be hard to fix.
9:19:03
beach
I think I am on the right track. I think I just identified some more specialized code that I can eliminate if I do it this way.
9:27:43
beach
Bad news: It is currently a mess. Good news: When I clean it up, it will be much prettier.
Monday, 15th of October 2018, 20:47:57 UTC