Skip to content

Commit

Permalink
1.0.44.29: full warnings for duplicate CASE keys during SBCL build
Browse files Browse the repository at this point in the history
  ...and fix the issue revealed.

  Thanks to Cyrus Harmon for the heads-up.
  • Loading branch information
nikodemus committed Nov 18, 2010
1 parent caab09a commit b29df9f
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
3 changes: 2 additions & 1 deletion src/code/macros.lisp
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,8 @@ invoked. In that case it will store into PLACE and start over."

(eval-when (#-sb-xc :compile-toplevel :load-toplevel :execute)

(define-condition duplicate-case-key-warning (style-warning)
;;; Make this a full warning during SBCL build.
(define-condition duplicate-case-key-warning (#-sb-xc-host style-warning #+sb-xc-host warning)
((key :initarg :key
:reader case-warning-key)
(case-kind :initarg :case-kind
Expand Down
8 changes: 4 additions & 4 deletions src/code/numbers.lisp
Original file line number Diff line number Diff line change
Expand Up @@ -855,10 +855,10 @@ the first."
;; conversion.
(multiple-value-bind (lo hi)
(case '(dispatch-type y)
('single-float
(single-float
(values most-negative-exactly-single-float-fixnum
most-positive-exactly-single-float-fixnum))
('double-float
(double-float
(values most-negative-exactly-double-float-fixnum
most-positive-exactly-double-float-fixnum)))
(if (<= lo y hi)
Expand All @@ -872,10 +872,10 @@ the first."
;; Likewise
(multiple-value-bind (lo hi)
(case '(dispatch-type x)
('single-float
(single-float
(values most-negative-exactly-single-float-fixnum
most-positive-exactly-single-float-fixnum))
('double-float
(double-float
(values most-negative-exactly-double-float-fixnum
most-positive-exactly-double-float-fixnum)))
(if (<= lo y hi)
Expand Down
2 changes: 1 addition & 1 deletion version.lisp-expr
Original file line number Diff line number Diff line change
Expand Up @@ -20,4 +20,4 @@
;;; checkins which aren't released. (And occasionally for internal
;;; versions, especially for internal versions off the main CVS
;;; branch, it gets hairier, e.g. "0.pre7.14.flaky4.13".)
"1.0.44.28"
"1.0.44.29"

0 comments on commit b29df9f

Please sign in to comment.