quotient-im / libQuotient

A Qt library to write cross-platform clients for Matrix
https://quotient-im.github.io/libQuotient/
GNU Lesser General Public License v2.1
134 stars 56 forks source link

Expose 3PID request fields #684

Closed portaloffreedom closed 1 year ago

portaloffreedom commented 1 year ago

Mandatory fields are now exposed, make it impossible to send RequestTokenTo3PIDEmailJob and RequestTokenTo3PIDMSISDNJob

KitsuneRal commented 1 year ago

Which actual problem are you trying to solve? The proposed changes don't change anything. From the respective page at cppreference.com:

If access-specifier is omitted, it defaults to public for classes declared with class-key struct and to private for classes declared with class-key class.

portaloffreedom commented 1 year ago

ah! you are right, I was too quick on this. I was trying to do this:

    Quotient::EmailValidationData req {
         .email = email
    };

because I like the compact style. Today I learned two things:

KitsuneRal commented 1 year ago

Yeah, I was suspecting you tried to use designated initialisers with those structures. It is terribly aggravating that they let this use case fall through the C++ standard cracks.