Page MenuHomePhabricator

csharp: Suppressing CA1051 for structs.
Needs RevisionPublic

Authored by brunobelo on Oct 29 2019, 7:51 AM.

Diff Detail

Repository
rEFL core/efl
Branch
ca1051
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 14246
Build 9821: arc lint + arc unit
brunobelo created this revision.Oct 29 2019, 7:51 AM
brunobelo requested review of this revision.Oct 29 2019, 7:51 AM
lauromoura requested changes to this revision.Oct 31 2019, 3:08 AM

Marking as request changes while we discuss the proper approach in the task. :)

This revision now requires changes to proceed.Oct 31 2019, 3:08 AM

Marking as request changes while we discuss the proper approach in the task. :)

Well, after discussion, we can change the approach to make the public struct fields use properties instead of suppressing the warning.

After D10588, the nested native structs are not public anymore so this should not be a problem for them.