freenode/#clim - IRC Chatlog
Search
13:53:33
jackdaniel
clim spec doesn't provide a way to change list-pane-items, right? (I mean - we have that implemented in climi - shouldn't we provide it in extension or something?)
14:07:27
slyrus
jackdaniel: alexandria has ensure-list, but it does something different. I'll rename my function. Mine takes a sequence and returns it unchanged if it's a list, or makes a new list with the contents of the sequence
14:08:48
jackdaniel
OK, please rename it then (apparently I've got confused, small comment there what it does (and how it's different) will be nice - docstring maybe?)
14:20:27
jackdaniel
slyrus: will it be a problem if I'll assign my own pull requests for review to you? even brief peer review improves code quality and increases confidence that the change is OK
14:23:26
slyrus
so this +nowhere+ thing is interesting. something must have changed in the design/region initialization sequence. we reference these regions from design.lisp, even though region is a design and region.lisp gets loaded after design.lisp. what we were doing before was wrong, but I could just forward reference +everywhere+ (which was actually my first "fix" for this).
14:24:21
slyrus
I may have been too clever by half in trying to keep +everywhere+ as a constant-ish thing. It's a little disconcerting to lie to the users and just have everywhere be a normal (special) variable, not a constant, but it seems we do that anyway.
15:06:44
slyrus
no PR for you. now the only bona fide warnings are the %make-image-sheet-get-function thing I filed an Issue on and Drei, which I assume we won't fix.
15:10:20
beach
Pretty sure Drei is fixable. Athas did a lot of work to separate (first) Climacs into Drei and the rest of the stuff.
15:11:36
jackdaniel
btw, I think it's a kludge, that Drei depends on ESA, which depends on clim-core, but text-editor-substrate depends on Drei
15:13:08
jackdaniel
it's not a big problem. It's a kludge, because we have a Library, which depends on McCLIM, but McCLIM depends on that library too. That makes simplifying dep graph quite impossible with this regard
15:16:05
jackdaniel
yes, I agree. my point is that we can't have two systems: mcclim/core and mcclim/backend where first has two modules, "basic" and "core" and the second picks the most appropriate backend for a platform, because between "basic" and "core" there is Drei - that's why I've said kludge
15:16:40
jackdaniel
now we have clim-basic, esa, drei, clim-core, clim systems, which make a core altogether
15:25:13
slyrus
1. &optional and &key specified in the same lambda list, which I assume we won't fix
16:21:05
jackdaniel
beach: could you add slyrus to project contributors? I'm not able to assign reviews to him (had to mention him in comment)