18:14:39jackdanielregarding the PR against McCLIM I've asked for few changes
18:21:11ck_jackdaniel: ok. I don't understand the name change request: the function doesn't 'ensure' the value. It buffers or caches it.
18:22:27jackdanielas 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:02ck_I simply wasn't aware of any such convention, and was going by name alone. Is there some documentation I can look up?
18:23:20jackdanielensure-gethash is a function in alexandria
18:25:07ck_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:25:42jackdanielas I said, if this feels inadequate to you just ignore this remark
19:37:42frgojackdaniel: 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:56jackdanielfrgo: I'll look into the def-clx-class thing tomorrow and we will compare notes, hm?
21:01:25jackdanielfrgo: 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:24jackdanielfrom the bright side changing it in this single place to defstruct should not impose any problem