[racket] Unreachable line reached

>> ;;;;;;;;;;;;;;;;;;;;;;;;;;;;
>> (for ([line (in-lines in)])
>>    (match (string-split line ",")
>>      [(list path modtime-str md5-str)
>>       (define modtime (string->number (string-trim modtime-str)))
>>       (define md5 (string->bytes/utf-8 (string-trim md5-str)))
>
> why not let for these?
> I have always heard these kind of define's would be poor style. What's your
> take on this?

Hi Paolo,


I prefer internal definitions now in professional code.  Internal
definitions are recommended by the Racket style guidelines:

http://www.ccs.neu.edu/home/matthias/Style/style/Choosing_the_Right_Construct.html#(part._.Definitions)