Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Plain View
Accumulo >> mail # dev >> Non Short Circuit Logic


+
David Medinets 2012-09-12, 16:29
Copy link to this message
-
Re: Non Short Circuit Logic
Judging from that code, p is never null so that second case is unnecessary.

Sent from my phone, so pardon the typos and brevity.
On Sep 12, 2012 12:29 PM, "David Medinets" <[EMAIL PROTECTED]> wrote:

> The if statement belows uses a single & instead of &&. Therefore it is
> not using short-circut logic. This seems like a typo. Does anyone
> object if I change the '&' to '&&'?
>
>     String delim = "";
>     shellState.getReader().printString("System permissions: ");
>     for (SystemPermission p : SystemPermission.values()) {
>       if
> (shellState.getConnector().securityOperations().hasSystemPermission(user,
> p) & p != null) {
>         shellState.getReader().printString(delim + "System." + p.name());
>         delim = ", ";
>       }
>     }
>     shellState.getReader().printNewline();
>
+
Jim Klucar 2012-09-12, 18:04
+
David Medinets 2012-09-12, 20:54
+
John Vines 2012-09-12, 21:00
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB