Skip to content

Commit c7785e8

Browse files
authored
Merge pull request #1404 from herbie-fp/autofix-50-1
Automated Resyntax fixes
2 parents eb31bdd + 576f2b3 commit c7785e8

File tree

3 files changed

+16
-13
lines changed

3 files changed

+16
-13
lines changed

src/api/server.rkt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@
236236
job-id]
237237
[(list 'count) (hash-count queued-jobs)]
238238
[(list 'improve)
239-
(for/list ([(job-id result) (in-hash completed-jobs)]
239+
(for/list ([result (in-hash-values completed-jobs)]
240240
#:when (equal? (hash-ref result 'command) "improve"))
241241
result)]
242242
[(list 'forget job-id)
@@ -368,7 +368,7 @@
368368
; Retrieve the improve results for results.json
369369
[(list 'improve handler)
370370
(define improved-list
371-
(for/list ([(job-id result) (in-hash completed-jobs)]
371+
(for/list ([result (in-hash-values completed-jobs)]
372372
#:when (equal? (hash-ref result 'command) "improve"))
373373
result))
374374
(place-channel-put handler improved-list)]

src/core/batch.rkt

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -180,12 +180,10 @@
180180
(λ (brf recurse)
181181
(define node (deref brf))
182182
(define node* (f node))
183-
(define brf*
184-
(let loop ([node* node*])
185-
(match node*
186-
[(? batchref? brf) (recurse brf)]
187-
[_ (batch-push! out (expr-recurse node* (compose batchref-idx loop)))])))
188-
brf*)))
183+
(let loop ([node* node*])
184+
(match node*
185+
[(? batchref? brf) (recurse brf)]
186+
[_ (batch-push! out (expr-recurse node* (compose batchref-idx loop)))])))))
189187

190188
;; Allocates new batch
191189
(define (batch-apply b brfs f)

src/core/egg-herbie.rkt

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -666,10 +666,11 @@
666666
(vector-set! id->eclass id (cons enode* (vector-ref id->eclass id)))
667667
(match enode*
668668
[(list _ ids ...)
669-
(if (null? ids)
670-
(vector-set! id->leaf? id #t)
671-
(for ([child-id (in-list ids)])
672-
(vector-set! id->parents child-id (cons id (vector-ref id->parents child-id)))))]
669+
#:when (null? ids)
670+
(vector-set! id->leaf? id #t)]
671+
[(list _ ids ...)
672+
(for ([child-id (in-list ids)])
673+
(vector-set! id->parents child-id (cons id (vector-ref id->parents child-id))))]
673674
[(? symbol?) (vector-set! id->leaf? id #t)]
674675
[(? number?) (vector-set! id->leaf? id #t)]))))
675676

@@ -1108,7 +1109,11 @@
11081109

11091110
;; Is fractional with odd denominator.
11101111
(define (fraction-with-odd-denominator? frac)
1111-
(and (rational? frac) (let ([denom (denominator frac)]) (and (> denom 1) (odd? denom)))))
1112+
(cond
1113+
[(rational? frac)
1114+
(define denom (denominator frac))
1115+
(and (> denom 1) (odd? denom))]
1116+
[else #f]))
11121117

11131118
;; Decompose an e-node representing an impl of `(pow b e)`.
11141119
;; Returns either `#f` or the `(cons b e)`

0 commit comments

Comments
 (0)