[racket-dev] Fix to module reader?

Following up: I haven't received any input on this yet.

On Fri, Mar 29, 2013 at 1:28 PM, Danny Yoo <dyoo at hashcollision.org> wrote:
> I'm running into an internal error with the syntax/module-reader
> library with the given test program, and wanted to double check with
> someone else before committing the fix upstream.
>
>
> The test file is:
>
>     https://gist.github.com/dyoo/5272982
>
>
> When I run this under an unpatched module reader, I see the following error:
>
> 128-110-73-138:Desktop dyoo$ raco test prepend-lang-line.rkt
> testing prepend-lang-line.rkt
> -: contract violation
>   expected: number?
>   given: #f
>   argument position: 2nd
>   other arguments...:
>    16
>   context...:
>    /Applications/Racket
> v5.3.2/collects/syntax/module-reader.rkt:173:2: wrap-internal
>    wrap-internal/wrapper
>    /Users/dyoo/Desktop/prepend-lang-line.rkt: [running body]
>    /Applications/Racket v5.3.2/collects/compiler/commands/test.rkt:
> [running body]
>    /Applications/Racket v5.3.2/collects/raco/raco.rkt: [running body]
>    /Applications/Racket v5.3.2/collects/raco/main.rkt: [running body]
>
>
>
>
> My proposed patch is:
>
>
> https://github.com/dyoo/racket/commit/9322570e3e0210b99ec8a85dca52b62b4dde66fd
>
>
>
>
> Does this look ok?  Also, if I do push this upstream, how do I attach
> my test file so it gets run as part of the DrDr regression suite?