Member variables of type int or long are auto initialized to 0 in Java (as
opposed to an unknown value).

That being said, that might not be a good default value for this situation.

On Tuesday, September 1, 2015, DaanHoogland <g...@git.apache.org> wrote:

> Github user DaanHoogland commented on a diff in the pull request:
>
>     https://github.com/apache/cloudstack/pull/766#discussion_r38394939
>
>     --- Diff:
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
> ---
>     @@ -258,7 +258,7 @@
>
>          private String _updateHostPasswdPath;
>
>     -    private int _dom0MinMem;
>     +    private long _dom0MinMem;
>     --- End diff --
>
>     I think we need to initialize to a reasonable default or 0L. Not sure
> if it is possible by installation but otherwise the properties file might
> be corrupted or empty.
>
>
> ---
> If your project is set up for it, you can reply to this email and have your
> reply appear on GitHub as well. If your project does not have this feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastruct...@apache.org <javascript:;> or
> file a JIRA ticket
> with INFRA.
> ---
>


-- 
*Mike Tutkowski*
*Senior CloudStack Developer, SolidFire Inc.*
e: mike.tutkow...@solidfire.com
o: 303.746.7302
Advancing the way the world uses the cloud
<http://solidfire.com/solution/overview/?video=play>*™*

Reply via email to