Page MenuHomePhabricator

csharp: klass style changes
AcceptedPublic

Authored by lauromoura on Mon, Apr 15, 3:11 PM.

Details

Summary

Warnings down to 120k

Depends on D8612

Diff Detail

Repository
rEFL core/efl
Branch
csharp_conventions (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 10856
Build 8442: arc lint + arc unit
lauromoura created this revision.Mon, Apr 15, 3:11 PM
lauromoura requested review of this revision.Mon, Apr 15, 3:11 PM
lauromoura edited the summary of this revision. (Show Details)
lauromoura added a reviewer: segfaultxavi.
lauromoura edited projects, added efl: language bindings; removed efl.
vitor.sousa accepted this revision.Tue, Apr 16, 8:09 AM
This revision is now accepted and ready to land.Tue, Apr 16, 8:09 AM
segfaultxavi accepted this revision.Tue, Apr 16, 10:26 AM

Docs look fine. They also render correctly, including links.

src/bin/eolian_mono/eolian/mono/klass.hh
236

I know it's unrelated but the last "" should be "\n" or this whole line should be removed when there's no root. Otherwise whitespace is wrong :P