From: Tobias C. R. <tc...@fr...> - 2009-01-11 16:56:05
|
"Nikodemus Siivola" <nik...@ra...> writes: > Attached patch is my approach to this: > > The intention is that users can either define appropriate > SOURCE-LOCATION methods for their objects, or just inherit from the > SOURCE-LOCATION-MIXIN when appropriate. > > * SB-EXT:CURRENT-SOURCE-LOCATION is a macro that does what > SB-C:SOURCE-LOCATION used to do. > > * SB-EXT:SOURCE-LOCATION is a generic function users can add method > to, which returns the SB-EXT:SOURCE-LOCATION object corresponding > to the argument (previously called SB-C:DEFINITION-SOURCE-LOCATION > object.) I'm not convinced that the SB-C:DEFINITION-SOURCE-LOCATION should be available to the user as it isn't clear how it's supposed to relate to SB-INTROSPECT. Instead the stuff from SB-INTROSPECT should be used exclusively. I.e. make the above available from SB-INTROSPECT and make it work on SB-INTROSPECT:DEFINITION-SOURCE objects. -T. |