Change in ovirt-engine-sdk-ruby[master]: Implement == and eql? for types

Juan Hernandez has submitted this change and it was merged. Change subject: Implement == and eql? for types ...................................................................... Implement == and eql? for types This patch changes the SDK so that struct types implement the '==' operator and the 'eql?' methods comparing the values of the instance variables. For example, the '==' operator generated for the 'ExternalVmImport' type is implemented like this: # # Returns `true` if `self` and `other` have the same attributes and # values. # def ==(other) super && @cluster == other.cluster && @cpu_profile == other.cpu_profile && @drivers_iso == other.drivers_iso && @host == other.host && @name == other.name && @password == other.password && @provider == other.provider && @quota == other.quota && @sparse == other.sparse && @storage_domain == other.storage_domain && @url == other.url && @username == other.username && @vm == other.vm end Change-Id: I945b79ba08fb9add2b47f862962d7487c1378868 Bug-Url: https://bugzilla.redhat.com/1383613 Signed-off-by: Juan Hernandez <juan.hernandez@redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/TypesGenerator.java M sdk/lib/ovirtsdk4/type.rb M sdk/spec/type_spec.rb 3 files changed, 225 insertions(+), 21 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/65340 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I945b79ba08fb9add2b47f862962d7487c1378868 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez@redhat.com> Gerrit-Reviewer: Martin Betak <mbetak@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
juan.hernandez@redhat.com