Models: using default methods for builder methods?

[jmini]

In the org.eclipse.microprofile.openapi.models package and sub-packages, the builder-pattern is used:

    T getXyzAbcd();

    void setXyzAbcd(T xyzAbcd);

    U ayzAbcd(T xyzAbcd);

The ayzAbcd(T) method is just a builder method.

Could this method be a default method?

    default U XyzAbcd(T xyzAbcd) {
        setXyzAbcd(xyzAbcd);
        return this;
    }

In my opinion it would ease the work of people who wants to create an implementation of the model interfaces.

Or is there a reason to not do this?

[arthurdm]

We agreed with this in today's hangout. This issue will be used to drive the proposed work of adding the default impls for these simple builder methods.

[jmini]

I have created an example in #242 for the Operation class. Before I extend this pattern to all builders methods, I want to be sure that this is the idea we have discussed here.

[jmini]

This issue will be used to drive the proposed work of adding the default impls for these simple builder methods.

Have you other changes than the one in #242 in mind?

[arthurdm]

that was all. thanks again @jmini!