[mcclim-devel] Possible bug

[Andy Hefner]

Ah, right. I guess the lisps I use don't check that type declaration,
so I never noticed. Nice work.


On Sun, 20 Mar 2005 22:40:27 -0800, David Christiansen
<chri0665 at uidaho.edu> wrote:
> I've tracked down what seems to be the spot....
> The defclass for meta-list-pane contains the following slot specifier:
> (mode        :initarg :mode
>                  :initform :exclusive
>                  :reader list-pane-mode
>                  :type (member :one-of :some-of))
> Perhaps the :type should read (member :one-of :some-of :exclusive
> :nonexclusive) instead?
> It looks like list-pane-exclusive-p already handles the options listed
> in the spec.
> 
> ;;------------------------------------------------------------------
> ;; -David Christiansen
> ;;
> 
> 
> On 20 Mar, 2005, at 18:39, Andy Hefner wrote:
> 
> > Strangely enough the vendor CLIMs call these keywords :some-of and
> > :one-of, despite what the spec says. My intent was to support both,
> > not sure why this isnt't working..
> >
> >
> >
> > On Sun, 20 Mar 2005 16:40:54 -0800, David Christiansen
> > <chri0665 at uidaho.edu> wrote:
> >> Hello all, I'm new to the list.  I've been writing a little Connect 4
> >> clone with McClim and I think I found a place where the implementation
> >> diverges from the standard.  I'm using the Mothering Sunday release.
> >> When I try to (make-pane 'option-pane :value "foo" :mode :exclusive
> >> :items '("foo" "bar")), I get an error saying that :exclusive is not a
> >> member of type (or :some-of :one-of).  It looks to me as though the
> >> spec says that those options should be called :exclusive or
> >> :nonexclusive, though :one-of and :some-of seem to work just fine to
> >> me.
> >>
> >> The relevant section of the spec (chapter 30) can be seen at:
> >> http://www.mikemac.com/mikemac/clim/gadgets.html
> >>
> >> Am I reading it correctly?  If so, I'll try to grab a recent CVS and
> >> work up a patch later this week (though, being a newbie, I'd want
> >> someone to see it).  It seems localized enough that I should be able
> >> to
> >> figure it out.
> >>
> >> Thank you all for this handy library!
> > _______________________________________________
> > mcclim-devel mailing list
> > mcclim-devel at common-lisp.net
> > http://common-lisp.net/cgi-bin/mailman/listinfo/mcclim-devel
> >
> 
> _______________________________________________
> mcclim-devel mailing list
> mcclim-devel at common-lisp.net
> http://common-lisp.net/cgi-bin/mailman/listinfo/mcclim-devel
>