Commit 0490d52
Update coding conventions for C# 13 (#44283)
* Fix small open issues.
- #33341 already addressed in #36428.
- Fixes #37294: Add text that the shortened format is valid only when the runtime type matches the variable type.
- Fixes #37295: Don't use `ID` in the sample.
- Fixes #37296: Fix nullable warnings. Other issue comments are incorrect.
- Fixes #41748: Change the sample so the constructor is relevant.
- Fixes #42858: Add an explanation on declaring variables in top level statements.
* Remaining open issues
- Fixes #43838: Add notes for collection expressions and using primary constructors. Include naming recommendations.
- Fixes #43839 Update the overview of constructors to include information on primary constructors.
* Code review for style
Fix any style issues with the current code.
* Final edit pass
I also caught a couple sample bits that I'd neglected in the previous commit.
* Apply suggestions from code review
Co-authored-by: Genevieve Warren <[email protected]>
---------
Co-authored-by: Genevieve Warren <[email protected]>1 parent 462d811 commit 0490d52
File tree
10 files changed
+169
-138
lines changed- docs/csharp
- fundamentals/coding-style
- snippets/coding-conventions
- programming-guide/classes-and-structs
- snippets
- constructors
- using-constructors
- tutorials
10 files changed
+169
-138
lines changedLines changed: 46 additions & 23 deletions
Large diffs are not rendered by default.
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
| 6 | + | |
6 | 7 | | |
7 | 8 | | |
8 | 9 | | |
| |||
Lines changed: 65 additions & 57 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | | - | |
65 | | - | |
66 | | - | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
67 | 69 | | |
68 | 70 | | |
69 | 71 | | |
70 | 72 | | |
71 | 73 | | |
72 | 74 | | |
73 | | - | |
| 75 | + | |
74 | 76 | | |
75 | 77 | | |
76 | 78 | | |
| |||
123 | 125 | | |
124 | 126 | | |
125 | 127 | | |
126 | | - | |
127 | | - | |
128 | | - | |
129 | | - | |
130 | | - | |
131 | | - | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
132 | 131 | | |
133 | 132 | | |
134 | 133 | | |
| |||
152 | 151 | | |
153 | 152 | | |
154 | 153 | | |
155 | | - | |
156 | | - | |
157 | | - | |
158 | | - | |
| 154 | + | |
159 | 155 | | |
160 | 156 | | |
161 | 157 | | |
| |||
214 | 210 | | |
215 | 211 | | |
216 | 212 | | |
217 | | - | |
218 | | - | |
219 | | - | |
220 | | - | |
| 213 | + | |
221 | 214 | | |
222 | 215 | | |
223 | | - | |
224 | | - | |
225 | | - | |
226 | | - | |
| 216 | + | |
227 | 217 | | |
228 | 218 | | |
229 | 219 | | |
230 | | - | |
| 220 | + | |
231 | 221 | | |
232 | 222 | | |
233 | 223 | | |
234 | 224 | | |
235 | 225 | | |
236 | 226 | | |
237 | 227 | | |
238 | | - | |
239 | | - | |
| 228 | + | |
| 229 | + | |
240 | 230 | | |
241 | 231 | | |
242 | 232 | | |
243 | 233 | | |
244 | 234 | | |
245 | | - | |
246 | | - | |
247 | | - | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
248 | 238 | | |
249 | 239 | | |
250 | 240 | | |
251 | | - | |
| 241 | + | |
252 | 242 | | |
253 | 243 | | |
254 | 244 | | |
| |||
257 | 247 | | |
258 | 248 | | |
259 | 249 | | |
260 | | - | |
261 | | - | |
| 250 | + | |
| 251 | + | |
262 | 252 | | |
263 | 253 | | |
264 | 254 | | |
265 | | - | |
266 | | - | |
| 255 | + | |
| 256 | + | |
267 | 257 | | |
268 | 258 | | |
269 | 259 | | |
| |||
335 | 325 | | |
336 | 326 | | |
337 | 327 | | |
338 | | - | |
339 | | - | |
340 | | - | |
341 | | - | |
| 328 | + | |
342 | 329 | | |
343 | 330 | | |
344 | 331 | | |
345 | 332 | | |
346 | 333 | | |
347 | 334 | | |
348 | | - | |
349 | | - | |
350 | | - | |
351 | | - | |
| 335 | + | |
352 | 336 | | |
353 | 337 | | |
354 | 338 | | |
| |||
367 | 351 | | |
368 | 352 | | |
369 | 353 | | |
370 | | - | |
| 354 | + | |
371 | 355 | | |
372 | 356 | | |
373 | 357 | | |
374 | 358 | | |
375 | 359 | | |
376 | 360 | | |
377 | 361 | | |
378 | | - | |
379 | | - | |
380 | | - | |
381 | | - | |
382 | | - | |
383 | | - | |
384 | | - | |
385 | | - | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
386 | 369 | | |
387 | 370 | | |
388 | 371 | | |
389 | | - | |
| 372 | + | |
390 | 373 | | |
391 | 374 | | |
392 | 375 | | |
393 | 376 | | |
| 377 | + | |
394 | 378 | | |
395 | 379 | | |
396 | 380 | | |
397 | 381 | | |
398 | | - | |
| 382 | + | |
399 | 383 | | |
400 | | - | |
401 | | - | |
402 | | - | |
403 | | - | |
| 384 | + | |
| 385 | + | |
| 386 | + | |
| 387 | + | |
| 388 | + | |
| 389 | + | |
| 390 | + | |
| 391 | + | |
| 392 | + | |
404 | 393 | | |
405 | 394 | | |
406 | 395 | | |
| |||
459 | 448 | | |
460 | 449 | | |
461 | 450 | | |
462 | | - | |
| 451 | + | |
463 | 452 | | |
464 | 453 | | |
465 | 454 | | |
| |||
475 | 464 | | |
476 | 465 | | |
477 | 466 | | |
| 467 | + | |
| 468 | + | |
| 469 | + | |
| 470 | + | |
| 471 | + | |
| 472 | + | |
| 473 | + | |
| 474 | + | |
| 475 | + | |
| 476 | + | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
| 480 | + | |
| 481 | + | |
| 482 | + | |
| 483 | + | |
| 484 | + | |
| 485 | + | |
Lines changed: 15 additions & 20 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
| 12 | + | |
13 | 13 | | |
14 | | - | |
| 14 | + | |
15 | 15 | | |
16 | | - | |
| 16 | + | |
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | | - | |
| 23 | + | |
24 | 24 | | |
25 | | - | |
| 25 | + | |
26 | 26 | | |
27 | 27 | | |
28 | 28 | | |
29 | | - | |
| 29 | + | |
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | | - | |
| 33 | + | |
34 | 34 | | |
35 | 35 | | |
36 | 36 | | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
37 | 41 | | |
38 | 42 | | |
39 | | - | |
| 43 | + | |
40 | 44 | | |
41 | 45 | | |
42 | 46 | | |
| |||
48 | 52 | | |
49 | 53 | | |
50 | 54 | | |
51 | | - | |
52 | | - | |
53 | | - | |
54 | | - | |
55 | | - | |
56 | | - | |
57 | | - | |
58 | | - | |
59 | 55 | | |
60 | 56 | | |
61 | 57 | | |
62 | | - | |
63 | | - | |
| 58 | + | |
64 | 59 | | |
Lines changed: 13 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
| 28 | + | |
32 | 29 | | |
33 | 30 | | |
34 | 31 | | |
| |||
63 | 60 | | |
64 | 61 | | |
65 | 62 | | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
0 commit comments