> + if (options.getRunScript() != null) {
> + builder.add(options.getRunScript());
> + }
> + if (options.getPrivateKey() != null) {
> + builder.add(new InstallRSAPrivateKey(options.getPrivateKey()));
> + }
> +
> + ImmutableList<Statement> bootstrap = builder.build();
> + if (!bootstrap.isEmpty()) {
> + if (options.getTaskName() == null && !(options.getRunScript()
> instanceof InitScript)) {
> + options.nameTask("bootstrap");
> + }
> + return bootstrap.size() == 1 ? bootstrap.get(0) : new
> StatementList(bootstrap);
> + }
> +
> + return null;
Just as a personal style preference, handle the "exceptional case" first? I.e.
```
if (bootstrap.isEmpty()) {
return null;
}
...
```
?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/70/files#r17825999
