[pve-devel] [PATCH 2/7] Storage Plugins: extend clone_image by snap parameter and add support to RBDPlugin

[Stefan Priebe]

Am 07.02.2014 16:09, schrieb Dietmar Maurer:
>>>
>>> "if $snapname" should be enough.
>>
>> If $snapname is "0mysnap" it isn't as perl would interpret this as false
>
> I don't really remember, but normally our names are not allowed to start with numbers.
> No sure if we have that restrictions for snapshot names.

I just thought it's safer and doesn't hurt. Also it does not give any 
warnings for undef values - so it's a pretty good way to check for valid 
values in those cases.

Stefan