forked fromfsharp/fsharp
- Notifications
You must be signed in to change notification settings - Fork0
Commit5c491b2
Improve warning for ignored values in sequence expressions - fixes #4228 (#4245)
* Improve warning for ignored values in sequence expressions - fixes #4228* Adding a test* Show the type* Add a case for when we need to suggest yield!* Check sequence type* Manually copy resx and fs from txt conversion* fix rebase issue* Only report yield! for sequences1 parentac2d8d5 commit5c491b2
File tree
25 files changed
+1478
-1253
lines changed- src
- buildfromsource/FSharp.Compiler.Private
- fsharp
- xlf
- tests/fsharpqa/Source/Warnings
25 files changed
+1478
-1253
lines changedLines changed: 1243 additions & 1239 deletions
Large diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
597 | 597 | | |
598 | 598 | | |
599 | 599 | | |
600 | | - | |
601 | | - | |
602 | | - | |
603 | 600 | | |
604 | 601 | | |
605 | 602 | | |
| |||
4306 | 4303 | | |
4307 | 4304 | | |
4308 | 4305 | | |
| 4306 | + | |
| 4307 | + | |
| 4308 | + | |
| 4309 | + | |
| 4310 | + | |
| 4311 | + | |
4309 | 4312 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
699 | 699 | | |
700 | 700 | | |
701 | 701 | | |
702 | | - | |
| 702 | + | |
703 | 703 | | |
704 | 704 | | |
705 | 705 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
142 | 142 | | |
143 | 143 | | |
144 | 144 | | |
| 145 | + | |
| 146 | + | |
145 | 147 | | |
146 | 148 | | |
147 | 149 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
73 | | - | |
| 73 | + | |
| 74 | + | |
74 | 75 | | |
75 | 76 | | |
76 | 77 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1422 | 1422 | | |
1423 | 1423 | | |
1424 | 1424 | | |
| 1425 | + | |
| 1426 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
717 | 717 | | |
718 | 718 | | |
719 | 719 | | |
720 | | - | |
| 720 | + | |
| 721 | + | |
721 | 722 | | |
722 | 723 | | |
723 | 724 | | |
| |||
830 | 831 | | |
831 | 832 | | |
832 | 833 | | |
833 | | - | |
| 834 | + | |
| 835 | + | |
834 | 836 | | |
835 | 837 | | |
836 | 838 | | |
837 | 839 | | |
838 | 840 | | |
839 | 841 | | |
840 | 842 | | |
841 | | - | |
842 | | - | |
843 | | - | |
844 | | - | |
845 | | - | |
| 843 | + | |
| 844 | + | |
| 845 | + | |
| 846 | + | |
| 847 | + | |
| 848 | + | |
| 849 | + | |
| 850 | + | |
| 851 | + | |
| 852 | + | |
| 853 | + | |
| 854 | + | |
| 855 | + | |
| 856 | + | |
| 857 | + | |
| 858 | + | |
| 859 | + | |
| 860 | + | |
| 861 | + | |
| 862 | + | |
846 | 863 | | |
847 | 864 | | |
848 | 865 | | |
| |||
5567 | 5584 | | |
5568 | 5585 | | |
5569 | 5586 | | |
5570 | | - | |
| 5587 | + | |
5571 | 5588 | | |
5572 | 5589 | | |
5573 | 5590 | | |
| |||
8255 | 8272 | | |
8256 | 8273 | | |
8257 | 8274 | | |
| 8275 | + | |
8258 | 8276 | | |
8259 | 8277 | | |
8260 | 8278 | | |
| |||
10413 | 10431 | | |
10414 | 10432 | | |
10415 | 10433 | | |
10416 | | - | |
| 10434 | + | |
10417 | 10435 | | |
10418 | 10436 | | |
10419 | 10437 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6972 | 6972 | | |
6973 | 6973 | | |
6974 | 6974 | | |
| 6975 | + | |
| 6976 | + | |
| 6977 | + | |
| 6978 | + | |
| 6979 | + | |
| 6980 | + | |
| 6981 | + | |
| 6982 | + | |
| 6983 | + | |
| 6984 | + | |
6975 | 6985 | | |
6976 | 6986 | | |
6977 | 6987 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6972 | 6972 | | |
6973 | 6973 | | |
6974 | 6974 | | |
| 6975 | + | |
| 6976 | + | |
| 6977 | + | |
| 6978 | + | |
| 6979 | + | |
| 6980 | + | |
| 6981 | + | |
| 6982 | + | |
| 6983 | + | |
| 6984 | + | |
6975 | 6985 | | |
6976 | 6986 | | |
6977 | 6987 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
6972 | 6972 | | |
6973 | 6973 | | |
6974 | 6974 | | |
| 6975 | + | |
| 6976 | + | |
| 6977 | + | |
| 6978 | + | |
| 6979 | + | |
| 6980 | + | |
| 6981 | + | |
| 6982 | + | |
| 6983 | + | |
| 6984 | + | |
6975 | 6985 | | |
6976 | 6986 | | |
6977 | 6987 | | |
0 commit comments
Comments
(0)