Skip to content

Commit 6288b07

Browse files
committed
Fix self-lint suggestions
1 parent 16dd37d commit 6288b07

3 files changed

Lines changed: 9 additions & 9 deletions

File tree

src/FSharpLint.Core/Framework/Ast.fs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -322,16 +322,16 @@ module Ast =
322322
| SynExpr.Downcast(expression, synType, _) ->
323323
addMany [Type synType; Expression expression]
324324
// regular let or use
325-
| SynExpr.LetOrUse letOrUse when letOrUse.IsBang = false ->
326-
add <| Expression letOrUse.Body
327-
List.revIter (Binding >> add) letOrUse.Bindings
325+
| ExpressionUtilities.LetOrUse({Bindings = bindings; Body = body}, false, _) ->
326+
add <| Expression body
327+
List.revIter (Binding >> add) bindings
328328
// let! or use!
329-
| SynExpr.LetOrUse letOrUse when letOrUse.IsBang = true -> //(_, _, _, true, bindings, leftHandSide, _, _) ->
330-
match letOrUse.Bindings with
329+
| ExpressionUtilities.LetOrUse({Bindings = bindings; Body = body}, true, _) ->
330+
match bindings with
331331
| firstBinding :: andBangs ->
332332
match firstBinding with
333333
| SynBinding(headPat = pattern; expr = rightHandSide) ->
334-
addMany [Expression rightHandSide; Expression letOrUse.Body]
334+
addMany [Expression rightHandSide; Expression body]
335335
List.iter (fun (SynBinding(headPat = pattern; expr = body)) ->
336336
addMany [Expression body; Pattern pattern]
337337
) andBangs

src/FSharpLint.Core/Rules/Conventions/FavourNonMutablePropertyInitialization.fs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ let rec private processLetBinding (instanceNames: Set<string>) (body: SynExpr) (
6464
and [<TailCall>] processExpression (expression: SynExpr) (continuation: unit -> array<WarningDetails>) : array<WarningDetails> =
6565
Array.append
6666
(match expression with
67-
| SynExpr.LetOrUse letOrUse when letOrUse.IsBang = false ->
67+
| SynExpr.LetOrUse letOrUse when not letOrUse.IsBang ->
6868
let instanceNames = extraFromBindings letOrUse.Bindings List.Empty |> Set.ofList
6969
processLetBinding instanceNames letOrUse.Body returnEmptyArray
7070
| SynExpr.Sequential(_, _, expr1, expr2, _, _) ->

src/FSharpLint.Core/Rules/Smells/AsyncExceptionWithoutReturn.fs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,8 @@ let rec checkExpression (expression: SynExpr) (range: range) (continuation: unit
5555
}
5656
| SynExpr.App (_, _, funcExpr, _, innerRange) ->
5757
checkExpression funcExpr innerRange returnEmptyArray
58-
| SynExpr.LetOrUse letOrUse when letOrUse.IsBang = false ->
59-
checkExpression letOrUse.Body letOrUse.Range returnEmptyArray
58+
| ExpressionUtilities.LetOrUse({Body = body; Range = innerRange}, false, _) ->
59+
checkExpression body innerRange returnEmptyArray
6060
| _ -> Array.empty)
6161
(continuation ())
6262
and [<TailCall>] checkMultipleExpressions (expressions: list<SynExpr * range>) (continuation: unit -> array<WarningDetails>) =

0 commit comments

Comments
 (0)