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

Private val fix #28

Closed
wants to merge 4 commits into from
Closed

Private val fix #28

wants to merge 4 commits into from

Conversation

dsyme
Copy link
Contributor

@dsyme dsyme commented Jan 17, 2015

This is the fix for #27. Testing needs to be added.

@dsyme
Copy link
Contributor Author

dsyme commented Jan 17, 2015

Testing added, I believe this fix is now ready.

let test () =
A.x <- 1

module TestCompilerMessgeSet =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please fix the typo: TestCompilerMessageSet

@dsyme dsyme closed this in c5e6699 Jan 20, 2015
forki referenced this pull request in forki/visualfsharp Jan 21, 2015
fixes #27
resolves #28

commit bf3c9b3ff6151a09dd9319e5e54d21a0edd17c31
Merge: 5eb0657 7393d92
Author: latkin <[email protected]>
Date:   Mon Jan 19 16:57:58 2015 -0800

    Merge branch 'private-val-fix' of https://github.com/dsyme/visualfsharp into dsyme-private-val-fix

commit 7393d92
Author: Don Syme <[email protected]>
Date:   Sat Jan 17 11:50:54 2015 +0000

    add tests for val sets

commit 68555a3
Author: Don Syme <[email protected]>
Date:   Sat Jan 17 11:38:18 2015 +0000

    adjust range reported in error

commit bff938f
Author: Don Syme <[email protected]>
Date:   Sat Jan 17 11:35:08 2015 +0000

    fix compile

commit d2f12e4
Author: Don Syme <[email protected]>
Date:   Sat Jan 17 11:33:41 2015 +0000

    fix for private val set
dsyme pushed a commit to dsyme/fsharp that referenced this pull request Jul 25, 2017
fix deadlocking test by safeguarding the wait.
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

Successfully merging this pull request may close these issues.

2 participants