Change in ovirt-engine[master]: core: Exclude the ME_ENUM_FIELD_SETTER findbugs warning

[juan.hernandez at redhat.com]

Juan Hernandez has submitted this change and it was merged.

Change subject: core: Exclude the ME_ENUM_FIELD_SETTER findbugs warning
......................................................................


core: Exclude the ME_ENUM_FIELD_SETTER findbugs warning

Findbugs doesn't like that the OsRepositoryImpl.init method is public,
but we need that because it is the mechanism that we use to lazyly
initialize the OS repository.

This is the findbugs warning:

  Public enum method unconditionally sets its field

  This public method declared in public enum unconditionally sets enum
  field, thus this field can be changed by malicious code or by accident
  from another package. Though mutable enum fields may be used for lazy
  initialization, it's a bad practice to expose them to the outer world.
  Consider removing this method or declaring it package-private.

  Bug kind and pattern: ME - ME_ENUM_FIELD_SETTER

This patch changes the exclude filters for the "utils" project so that
this findbugs warning will be ignored.

Change-Id: Ibe37f727663fef92bc559edc7d2d665ccdf1aed5
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M backend/manager/modules/utils/exclude-filters.xml
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/46595
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibe37f727663fef92bc559edc7d2d665ccdf1aed5
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org