freenode/#clim - IRC Chatlog
Search
13:12:37
jackdaniel
text-selection in basic is about putting markings. it calls publish-selection which is defined in Backends/common/ports.lisp which implements the selection protocol (as in copy/paste) which in turn depends on presentation-translator machinery (so indeed clim-basic has a dependency on a module which depends on core and the problem is real); function publish-selection does not call things in core module
13:14:24
jackdaniel
that makes a good case that the text-selection protocol should be move to core; then maybe standard-output-stream should not depend on text-selection-mixin in favor of clim-stream-pane depending on it
18:21:11
ck_
jackdaniel: ok. I don't understand the name change request: the function doesn't 'ensure' the value. It buffers or caches it.
18:22:27
jackdaniel
as I understand ensure-* convention it is to query the value unless it is memoized (like i.e ensure-gethash), but if you feel that the new name would be worse then leave it as is
18:23:02
ck_
I simply wasn't aware of any such convention, and was going by name alone. Is there some documentation I can look up?
18:25:07
ck_
There's ensure-generic-function in the spec, but that doesn't quite behave as this drawable-depth thing behaves. ensure-generic-function ensures that the argument becomes a globally named function. (ensure-drawable-depth drawable) would do no such thing.
18:30:46
ck_
jackdaniel: thanks for including the history in the clx pull request. Let's hope it saves someone some time, sometime in the future
19:37:42
frgo
jackdaniel: I commented on your remarks. I will commit some changes based on your comments. The def-clx-class macro issue is a mystery to me.
19:53:56
jackdaniel
frgo: I'll look into the def-clx-class thing tomorrow and we will compare notes, hm?
21:01:25
jackdaniel
frgo: it seems that the issue is already grokked: it is because allegro uses defclass not defstruct. I still think that either both options should be removed from the def-clx-class for all implementations or extension should simply use defstruct
21:02:24
jackdaniel
from the bright side changing it in this single place to defstruct should not impose any problem