[Sofa.Core] minor refactoring for Data::read() to move into BaseData the reading code

[damienmarchal]

---

By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).

---

Reviewers will merge this pull-request only if

• it builds with SUCCESS for all platforms on the CI.
• it does not generate new warnings.
• it does not generate new unit test failures.
• it does not generate new scene test failures.
• it does not break API compatibility.
• it is more than 1 week old (or has fast-merge label).

[fredroy]

[ci-build][with-all-tests]

[damienmarchal]

Haha no surprise tests didn't worked :)

[damienmarchal]

Hi @alxbilger

Thanks for reviewing the PR. BaseData::read is an existing function in sofa, that do the reading from a stream into a data field.

The PR does not change only how it was implemented. Before the PR, Data::read is duplicated for every type of data stored in the data field while in most of the case the reading pattern is the same. With the PR the "shared" behavior is moved into the base class which allows to enforce a consistent behavior regarding error checking & resetting the value when the field to parse is empty.

[damienmarchal]

[ci-build][with-all-tests]

[damienmarchal]

Merged by mistake...reverted in emergency mode.