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.

Sign up for GitHub

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

Jump to bottom

Debug stepping behavior very poor for loops over arrays #12

Closed
KevinRansom opened this issue Jan 14, 2015 · 2 comments
Closed

Debug stepping behavior very poor for loops over arrays #12

KevinRansom opened this issue Jan 14, 2015 · 2 comments
Labels
Bug Impact-Medium (Internal MS Team use only) Describes an issue with moderate impact on existing code. Ready

Comments

@KevinRansom
Copy link
Member

Opened by: latkin

for loops over list seq and string all exhibit nice, consistent debug stepping behavior, but for loops over arrays don't work well at all. We should correct the debug range assignments here so that this is consistent.
19f2qwf

See example here (VS 2013, "just my code" off, "managed compat mode" disabled)

Loops over ranges are also inconsistent, but at least usable, so as a lower priority item we should fix those, too.
@KevinRansom KevinRansom self-assigned this Mar 25, 2015
@dsyme dsyme added the pri-2 label May 12, 2015
@dsyme dsyme added this to the VS 2015 milestone May 12, 2015
@dsyme dsyme assigned dsyme and unassigned KevinRansom May 12, 2015
@latkin latkin modified the milestones: vNext, VS 2015 Jun 10, 2015
@latkin
Copy link
Contributor

latkin commented Jun 10, 2015

Removing the VS 2015 milestone on this guy as it would not be approved by shiproom at this point (plus no fix imminent AFAIK).

@latkin latkin removed this from the vNext milestone Aug 4, 2015
@KevinRansom KevinRansom removed the pri-2 label Dec 4, 2015
@dsyme dsyme added Impact-Medium (Internal MS Team use only) Describes an issue with moderate impact on existing code. Area-Compiler labels Jan 8, 2016
@dsyme dsyme added the Ready label Jul 5, 2016
@dsyme dsyme mentioned this issue Jul 22, 2016
Merged
@dsyme
Copy link
Contributor

dsyme commented Jul 22, 2016

Fixed by above PR

@dsyme dsyme closed this as completed Jul 22, 2016
liboz pushed a commit to liboz/visualfsharp that referenced this issue Oct 17, 2016
@manofstick
Merge pull request dotnet#12 from liboz/manofstick
5f85d34 
making identity more efficient
nelak pushed a commit to nelak/visualfsharp that referenced this issue Nov 27, 2016
@AviAvni
Merge pull request dotnet#12 from nelak/fix-fsi-none-issue
72bc4fa 
Added more printing for tests for records
dsyme referenced this issue in dsyme/fsharp Feb 25, 2017
@forki
Merge pull request #12 from dsyme/forki-paket
3439d91 
packages at design time
kant2002 pushed a commit to kant2002/fsharp that referenced this issue Mar 17, 2023
@alfonsogarciacaro @ncave
Don't lose attributes of method parameters (dotnet#12)
ded9225 
Temporary fix, remove when upstream dotnet#13786 is fixed.
nojaf pushed a commit to nojaf/fsharp that referenced this issue Oct 25, 2023
@alfonsogarciacaro @ncave
Don't lose attributes of method parameters (dotnet#12)
3e59414 
Temporary fix, remove when upstream dotnet#13786 is fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Impact-Medium (Internal MS Team use only) Describes an issue with moderate impact on existing code. Ready
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@KevinRansom @latkin @dsyme