[
https://issues.apache.org/jira/browse/AVRO-3329?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17484448#comment-17484448
]
ASF subversion and git services commented on AVRO-3329:
-------------------------------------------------------
Commit 5eff9a7f503ab0de550c99b6b46ac628ca9d817d in avro's branch
refs/heads/master from Kyle Schoonover
[ https://gitbox.apache.org/repos/asf?p=avro.git;h=5eff9a7 ]
AVRO-3329 Added omitted braces to CodeGen class (#1480)
* AVRO-3329 Added omitted braces to CodeGen class
* revert names?.Contains null check
Co-authored-by: Kyle T. Schoonover <[email protected]>
> Add omitted braces in CodeGen class
> -----------------------------------
>
> Key: AVRO-3329
> URL: https://issues.apache.org/jira/browse/AVRO-3329
> Project: Apache Avro
> Issue Type: Improvement
> Components: csharp
> Affects Versions: 1.11.0
> Reporter: Kyle Schoonover
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> Styling improvement. It is standard that when there is an if statement that
> the following line of code is executed. However this can cause confusion
> when maintaining a code base. This would require all statements with
> optional braces to include the braces.
>
> Style Rule Reference: [StyleCopAnalyzers/SA1503.md at master ·
> DotNetAnalyzers/StyleCopAnalyzers
> (github.com)|https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1503.md]
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
