Skip to content

Conversation

@bennn
Copy link
Contributor

@bennn bennn commented Jul 14, 2022

add the two keyword args from:
racket/racket@cca0deb


Related, the error message for this is rough:

make-font.rkt:7:2: Type Checker: could not apply function;
 wrong number of arguments provided
  expected: 9
  given: 11
  in: (make-font #:size 1024.0 #:style style #:weight weight #:face font)
make-font.rkt:7:2: Type Checker: type mismatch
  expected: (U #<unsafe-undefined> Font-Smoothing)
  given: Positive-Float-No-NaN
  in: (make-font #:size 1024.0 #:style style #:weight weight #:face font)
make-font.rkt:7:2: Type Checker: type mismatch
  expected: (U #<unsafe-undefined> Font-Style)
  given: False
  in: (make-font #:size 1024.0 #:style style #:weight weight #:face font)
make-font.rkt:7:2: Type Checker: type mismatch
  expected: (U #<unsafe-undefined> Font-Weight)
  given: Font-Style
  in: (make-font #:size 1024.0 #:style style #:weight weight #:face font)

I think the best "fix" is to add lots more regression tests so errors like this in a base environment get caught by the TR CI.

@capfredf capfredf merged commit b1f9ced into racket:master Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants