[CSHARP-2982] Update bracing style Created: 28/Feb/20 Updated: 09/Mar/20 Resolved: 09/Mar/20 |
|
| Status: | Closed |
| Project: | C# Driver |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Task | Priority: | Trivial - P5 |
| Reporter: | Vincent Kam (Inactive) | Assignee: | Unassigned |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
Our current code style has opening braces on lines by themselves. At least two members of the team have expressed a preference to allow the opening brace to be part of the previous line of code e.g.
If a majority of the team prefers this style, perhaps we should consider using it. |
| Comments |
| Comment by Robert Stam [ 04/Mar/20 ] |
|
I sympathize with the request because I also personally favor K&R style with the opening braces on the same line. In fact, for the first few years of this project's life that was the brace style we used. No matter what style we use there will always be those who disagree with it. We decided years ago to switch to the Visual Studio defaults because that is what most of our community of users uses. Changing brace style (again) at this point would be counterproductive. We should just stick with the current style. |