core: Remove unused Tags methods

Change-Id: I8665db83c777876522612e595d7823b13f4b9aac
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: Make Event immutable

Removed unused setters from the Event class. Once an Event is created,
it makes no sense to modify it, and having the class immutable is a good
defensive practice for this kind of data objects.

Change-Id: I92eaad099f975d279fb1ab6a9c70112e7ea194d5
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: rm unused CLIParser.getNextParam()

Change-Id: I839f6afdb333fe23db2478687557b5c116e90292
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: rm unused VmDeviceCommonUtils.CDROM_IMAGE_ID

Change-Id: I3077f3edf6e7bf1268f4543fa66ce3631d89f57b
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: rm unused ObjectIdentityChecker methods

Change-Id: I3babf37b38921d06ec11c3a7a838edf246ed3f59
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: rm ObjectIdentityChecker.statusTypes member

This map is only updated, but is never queried. Therefore, it's just a
waste of memory, and should be removed.

Change-Id: I5460365ba54366697fbe72fc1b0066b8abea2d9a
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: rm useless ObjectIdentityChecker constructor

Since the previous patch removed the use of the status class, the
constructor accepting it has become useless, and could safely be
removed.

Change-Id: Iedea2389028858ff474db032adc4ed8a5e7677db
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: VmTemplateHandler.log modifiers

Made the log final, as per the proper Java conventions.

Change-Id: Ifb1868d3444ec5ff43681a1c45e95b71ab00171a
Signed-off-by: Allon Mureinik <amureini@redhat.com>
core: rename VmTemplateHandler methods

Renamed methods to adhere to proper Java naming conventions.

Change-Id: I8bbce4a8491ddbec3aec90655e7d7f2f3ab013d3
Signed-off-by: Allon Mureinik <amureini@redhat.com>