Page MenuHomePhabricator

csharp: Document eina_value.cs

Authored by lauromoura on Oct 8 2019, 5:38 PM.



Added more docs alongside the since information.

For items that should be skipped from the end user of the binding but
must stay public for generated code,
EditorBrowsable(EditorBrowsableState.Never) was used.

ref T8293

Diff Detail

rEFL core/efl
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
lauromoura created this revision.Oct 8 2019, 5:38 PM
lauromoura requested review of this revision.Oct 8 2019, 5:38 PM
segfaultxavi requested changes to this revision.Oct 9 2019, 1:01 AM

General comments:

  • Missing <param> and <returns> docs. I know adding ALL of them is a lot of work, but at least some of them are necessary for the user, for example, the return values of Compare or Set methods.
  • Copy and paste woes: Sets the contained valued. -> Sets the contained value.
  • Copy and paste woes: an Value -> a Value

Different how? I think this should be clarified... at least I do not know what optionals are and how they relate to Eina.Value.


Do not use acronyms.


What is the "target" value and what is an "optional"?


What does the returned bool mean?


whether this


different from



This revision now requires changes to proceed.Oct 9 2019, 1:01 AM
lauromoura updated this revision to Diff 25985.Oct 9 2019, 1:46 PM

Update after Xavi's comment.

lauromoura updated this revision to Diff 25986.Oct 9 2019, 1:58 PM
lauromoura marked 4 inline comments as done.

There was a rogue this

lauromoura marked 3 inline comments as done.Oct 9 2019, 2:00 PM
lauromoura updated this revision to Diff 25988.Oct 9 2019, 2:50 PM

Somehow the test was flaky. Removing it.

A few typos

segfaultxavi accepted this revision.Oct 10 2019, 1:18 AM

Just fixed some typos.

This revision is now accepted and ready to land.Oct 10 2019, 1:18 AM
This revision was automatically updated to reflect the committed changes.