Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate Find All References results after a closed and re-opened .NET Core 2.0 project #3557

Closed
vasily-kirichenko opened this issue Sep 6, 2017 · 3 comments
Milestone

Comments

@vasily-kirichenko
Copy link
Contributor

vasily-kirichenko commented Sep 6, 2017

  1. Clone and build this project: https://github.com/fable-compiler/fable-suave-scaffold
  2. Open ClientServer.sln.
  3. Observe that things work.
  4. Add a new file that defines the following:
    type private MaybeBuilder() = 
        member x.Bind(v, f) = Option.bind f v
        member x.Return(v) = Some v
        member x.Zero() = Some ()
        member x.Combine(v, f:unit -> _) = Option.bind f v
        member x.Delay(f : unit -> 'T) = f
        member x.Run(f) = f()
        member x.While(cond, f) =
            if cond() then x.Bind(f(), fun _ -> x.While(cond, f)) 
            else x.Zero()

    let private maybe = MaybeBuilder()

    let testFxn =
        let ret = 
            maybe {
                let! x = Some 1
                let! y = Some 2
                return x + y
            }
        ret
  1. Close ClientServer.sln.
  2. Re-open it.
  3. Find All References on a maybe.

You will see duplicate results in the Find All References window.

Old issue:
image

@cartermp
Copy link
Contributor

I cannot repro this on a .NET Core 2.0 project after #3564 in the 15.5 build.

no-repro

@cartermp cartermp changed the title Strange Find All References results on a CE builder on .NET Core 2.0 project Duplicate Find All References results after a closed and re-opened .NET Core 2.0 project Sep 12, 2017
@cartermp
Copy link
Contributor

Interestingly enough, this does repro after I close and reopen the solution.

@cartermp
Copy link
Contributor

This no longer reproduces. I believe all issues with duplicates have long been resolved.

@cartermp cartermp added this to the 15.6 milestone Aug 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants