Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Commit aca5753

Browse files
BigBadBleuCheesedanmoseley
authored andcommitted
Slight grammar improvement to coding-style.md (#28723)
i.e. was being used when it really should have been e.g. (see: https://www.grammarly.com/blog/know-your-latin-i-e-vs-e-g/)
1 parent 54a988a commit aca5753

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

Documentation/coding-guidelines/coding-style.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ The general rule we follow is "use Visual Studio defaults".
99

1010
1. We use [Allman style](http://en.wikipedia.org/wiki/Indent_style#Allman_style) braces, where each brace begins on a new line. A single line statement block can go without braces but the block must be properly indented on its own line and it must not be nested in other statement blocks that use braces (See issue [381](https://github.com/dotnet/corefx/issues/381) for examples).
1111
2. We use four spaces of indentation (no tabs).
12-
3. We use `_camelCase` for internal and private fields and use `readonly` where possible. Prefix instance fields with `_`, static fields with `s_` and thread static fields with `t_`. When used on static fields, `readonly` should come after `static` (i.e. `static readonly` not `readonly static`).
12+
3. We use `_camelCase` for internal and private fields and use `readonly` where possible. Prefix instance fields with `_`, static fields with `s_` and thread static fields with `t_`. When used on static fields, `readonly` should come after `static` (e.g. `static readonly` not `readonly static`).
1313
4. We avoid `this.` unless absolutely necessary.
14-
5. We always specify the visibility, even if it's the default (i.e.
15-
`private string _foo` not `string _foo`). Visibility should be the first modifier (i.e.
14+
5. We always specify the visibility, even if it's the default (e.g.
15+
`private string _foo` not `string _foo`). Visibility should be the first modifier (e.g.
1616
`public abstract` not `abstract public`).
1717
6. Namespace imports should be specified at the top of the file, *outside* of
1818
`namespace` declarations and should be sorted alphabetically.
@@ -23,8 +23,8 @@ The general rule we follow is "use Visual Studio defaults".
2323
Consider enabling "View White Space (Ctrl+E, S)" if using Visual Studio, to aid detection.
2424
9. If a file happens to differ in style from these guidelines (e.g. private members are named `m_member`
2525
rather than `_member`), the existing style in that file takes precedence.
26-
10. We only use `var` when it's obvious what the variable type is (i.e. `var stream = new FileStream(...)` not `var stream = OpenStandardInput()`).
27-
11. We use language keywords instead of BCL types (i.e. `int, string, float` instead of `Int32, String, Single`, etc) for both type references as well as method calls (i.e. `int.Parse` instead of `Int32.Parse`). See issue [391](https://github.com/dotnet/corefx/issues/391) for examples.
26+
10. We only use `var` when it's obvious what the variable type is (e.g. `var stream = new FileStream(...)` not `var stream = OpenStandardInput()`).
27+
11. We use language keywords instead of BCL types (e.g. `int, string, float` instead of `Int32, String, Single`, etc) for both type references as well as method calls (e.g. `int.Parse` instead of `Int32.Parse`). See issue [391](https://github.com/dotnet/corefx/issues/391) for examples.
2828
12. We use PascalCasing to name all our constant local variables and fields. The only exception is for interop code where the constant value should exactly match the name and value of the code you are calling via interop.
2929
13. We use ```nameof(...)``` instead of ```"..."``` whenever possible and relevant.
3030
14. Fields should be specified at the top within type declarations.

0 commit comments

Comments
 (0)