[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-3789?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17078892#comment-17078892
 ] 

Christopher Tubbs commented on ZOOKEEPER-3789:
----------------------------------------------

Cool! I'm not the only one who noticed this. :) Fixing all the javadoc warnings 
(and eventually, all non-javadoc compiler warnings, too) is something I have on 
my personal TODO list to fix soon. But, I'd be happy if somebody else got to it 
first. :)

> fix the build warnings about useless @return found by IDEA
> ----------------------------------------------------------
>
>                 Key: ZOOKEEPER-3789
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3789
>             Project: ZooKeeper
>          Issue Type: Improvement
>          Components: server
>            Reporter: maoling
>            Assignee: maoling
>            Priority: Minor
>
> 1. I use my IDEA to find these maven build warnings:
> {code:java}
> /Library/Java/JavaVirtualMachines/jdk1.8.0_191.jdk/Contents/Home/bin/java 
> -Dmaven.multiModuleProjectDirectory=/Users/maoling/workspaces/workspace_zookeeper/zookeeper
>  "-Dmaven.home=/Applications/IntelliJ 
> IDEA.app/Contents/plugins/maven/lib/maven3" 
> "-Dclassworlds.conf=/Applications/IntelliJ 
> IDEA.app/Contents/plugins/maven/lib/maven3/bin/m2.conf" 
> "-Dmaven.ext.class.path=/Applications/IntelliJ 
> IDEA.app/Contents/plugins/maven/lib/maven-event-listener.jar" 
> "-javaagent:/Applications/IntelliJ 
> IDEA.app/Contents/lib/idea_rt.jar=58545:/Applications/IntelliJ 
> IDEA.app/Contents/bin" -Dfile.encoding=UTF-8 -classpath 
> "/Applications/IntelliJ 
> IDEA.app/Contents/plugins/maven/lib/maven3/boot/plexus-classworlds-2.6.0.jar" 
> org.codehaus.classworlds.Launcher -Didea.version2019.3.1 -DskipTests=true 
> package -P !java-build
> {code}
> {code:java}
> Javadoc Warnings
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:340:
>  ?? - ??@see: 
> ?org.apache.zookeeper.ZooKeeper????getEphemerals(EphemeralsCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:340:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getEphemerals(String, 
> EphemeralsCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:271:
>  ?? - ??@link: ?org.apache.zookeeper.ZooKeeper????sync(String, VoidCallback, 
> Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:150:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getACL(String, Stat, 
> ACLCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:89:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getAllChildrenNumber(String, 
> AllChildrenNumberCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:202:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getChildren(String, boolean, 
> Children2Callback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:202:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getChildren(String, Watcher, 
> Children2Callback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:179:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getChildren(String, boolean, 
> ChildrenCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:179:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getChildren(String, Watcher, 
> ChildrenCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:227:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????create(String, byte[], List, 
> CreateMode, Create2Callback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:227:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????create(String, byte[], List, 
> CreateMode, Create2Callback, Object, long)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:121:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getData(String, boolean, 
> DataCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:121:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getData(String, Watcher, 
> DataCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:121:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getConfig(boolean, 
> DataCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:121:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????getConfig(Watcher, 
> DataCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:328:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????multi(Iterable, 
> MultiCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:70:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????exists(String, boolean, 
> StatCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:70:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????exists(String, Watcher, 
> StatCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:70:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????setData(String, byte[], int, 
> StatCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:70:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????setACL(String, List, int, 
> StatCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:262:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????create(String, byte[], List, 
> CreateMode, StringCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:299:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????delete(String, int, 
> VoidCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:299:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????removeAllWatches(String, 
> Watcher.WatcherType, boolean, VoidCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:299:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????removeWatches(String, 
> Watcher, Watcher.WatcherType, boolean, VoidCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:299:
>  ?? - ??@see: ?org.apache.zookeeper.ZooKeeper????sync(String, VoidCallback, 
> Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/AsyncCallback.java:299:
>  ?? - ??@link: ?org.apache.zookeeper.ZooKeeper????delete(String, int, 
> VoidCallback, Object)
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/ZKDatabase.java:513:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/util/CircularBuffer.java:70:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/FileTxnLog.java:318:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/FileTxnSnapLog.java:571:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/SnapStream.java:215:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/Util.java:264:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/persistence/Util.java:274:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Learner.java:94:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Leader.java:1201:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LearnerMaster.java:131:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Leader.java:265:
>  ?? - @param argument "qv," ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Leader.java:977:
>  ?? - @param argument "zxid," ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/Leader.java:977:
>  ?? - @param argument "sid," ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/LearnerHandler.java:1142:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/server/quorum/QuorumPeer.java:1949:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/cli/AclParser.java:37:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/cli/CliCommand.java:77:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/cli/CliCommand.java:85:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/cli/CliCommand.java:93:
>  ?? - @return ???????
> /Users/maoling/workspaces//zookeeper/zookeeper-server/src/main/java/org/apache/zookeeper/cli/CliCommand.java:118:
>  ?? - @return ???????
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to