Automated Resyntax fixes

[resyntax-ci]

This is an automated change generated by Resyntax.

Pass 1

Applied 1 fix to drracket/drracket/private/syncheck/blueboxes-gui.rkt

• Line 2, tidy-require: Keep imports in require sorted and grouped by phase, with collections before files.

Applied 12 fixes to drracket/drracket/private/syncheck/gui.rkt

• Line 21, tidy-require: Keep imports in require sorted and grouped by phase, with collections before files.
• Line 385, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 403, define-lambda-to-define: The define form supports a shorthand for defining functions (including function-returning functions).
• Line 2141, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2192, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2356, inline-unnecessary-define: This variable is returned immediately and can be inlined.
• Line 2464, define-lambda-to-define: The define form supports a shorthand for defining functions (including function-returning functions).
• Line 2612, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2640, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2650, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2726, cond-let-to-cond-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2755, flat-contract-migration: flat-contract is a legacy form for constructing contracts from predicates; predicates can be used directly as contracts now.

Applied 1 fix to drracket/drracket/private/syncheck/intf.rkt

• Line 3, tidy-require: Keep imports in require sorted and grouped by phase, with collections before files.

Pass 2

Applied 6 fixes to drracket/drracket/private/syncheck/gui.rkt

• Line 21, tidy-require: Keep imports in require sorted and grouped by phase, with collections before files.
• Line 387, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 403, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2090, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2588, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.
• Line 2598, let-to-define: Internal definitions are recommended instead of let expressions, to reduce nesting.

Summary

Fixed 20 issues in 3 files.

• Fixed 11 occurrences of let-to-define
• Fixed 4 occurrences of tidy-require
• Fixed 2 occurrences of define-lambda-to-define
• Fixed 1 occurrence of inline-unnecessary-define
• Fixed 1 occurrence of cond-let-to-cond-define
• Fixed 1 occurrence of flat-contract-migration

[jackfirth]

This diff is way too big. Filed jackfirth/resyntax#321 to add a limit for that.

[rfindler]

in class*, I'd like the keep the super class and the super interfaces on the first line and then have a newline. Is that possible @sorawee ?

[sorawee]

Totally! See sorawee/fmt#73

[rfindler]

I can't tell what's going on in the big diff. I'm not sure if this is a problem with the diff visualization algorithm on github or what.

[jackfirth]

Yeah this one's unreadable. I'm gonna close it and go implement a line limit.