If you are checking to see if the calendar is not set I'd prefer an explicit ```calendar is not None```. Did you consider assuming a standard calendar if calendar is None (in the same way as the ```iris.unit.Unit``` constructor?)