summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJustin Morey <jmorey@google.com>2015-08-11 17:24:12 -0500
committerJustin Morey <jmorey@google.com>2015-08-11 17:24:12 -0500
commit846e46861b23124b3360bac9bdc3ec587aaf60a5 (patch)
treeec814361b0b288ca7cc9090780073b1973235e0a
parent274a19e4c4a6dc90d1a71b5f5c54438f343868a5 (diff)
downloadandroid_external_doclava-846e46861b23124b3360bac9bdc3ec587aaf60a5.tar.gz
android_external_doclava-846e46861b23124b3360bac9bdc3ec587aaf60a5.tar.bz2
android_external_doclava-846e46861b23124b3360bac9bdc3ec587aaf60a5.zip
DO NOT MERGE Revert "Display annotations and annotation values when they are requested"
This breaks the GmsCore build, by (among other things) including private fields in our API definitions, which probably isn't right. This reverts commit 274a19e4c4a6dc90d1a71b5f5c54438f343868a5.
-rw-r--r--res/assets/templates/macros.cs35
-rw-r--r--src/com/google/doclava/AnnotationInstanceInfo.java16
-rw-r--r--src/com/google/doclava/MemberInfo.java15
3 files changed, 3 insertions, 63 deletions
diff --git a/res/assets/templates/macros.cs b/res/assets/templates/macros.cs
index 16711c4..eed8737 100644
--- a/res/assets/templates/macros.cs
+++ b/res/assets/templates/macros.cs
@@ -144,41 +144,6 @@ def:short_descr(obj) ?><?cs
/if ?><?cs
/def ?>
-
-<?cs
-# Show a list of annotations associated with obj
-#
-# pre is an HTML string to start the list.
-# between is an HTML string to include between each successive element.
-# post is an HTML string to end the list.
-# for example, call:show_annotations_list(cl, "<p>Annotations: ", "<br />", "</p>")
-# ?><?cs
-def:show_annotations_list(obj, pre, between, post) ?><?cs
- each:anno = obj.showAnnotations ?><?cs
- if:first(anno) ?><?cs
- var:pre ?><?cs
- /if ?>
- @<?cs var:anno.type.label ?>(<?cs
- each:elem = anno.elementValues ?><?cs
- var:elem.name ?>&nbsp;=&nbsp;<?cs
- var:elem.value ?><?cs
- if:last(elem) == 0 ?>, <?cs
- /if ?><?cs
- /each ?>)<?cs
- if:last(anno) == 0 ?><?cs
- var:between ?><?cs
- /if ?><?cs
- if:last(anno) ?><?cs
- var:post ?><?cs
- /if ?><?cs
- /each ?><?cs
-/def ?>
-
-<?cs # Show a comma-separated list of annotations associated with obj ?><?cs
-def:show_simple_annotations_list(obj, pre, post) ?><?cs
- call:show_annotations_list(obj, pre, ", ", post) ?><?cs
-/def ?>
-
<?cs # Show the red box with the deprecated warning ?><?cs
def:deprecated_warning(obj) ?><?cs
if:subcount(obj.deprecated) ?><p>
diff --git a/src/com/google/doclava/AnnotationInstanceInfo.java b/src/com/google/doclava/AnnotationInstanceInfo.java
index d353426..c72ca9a 100644
--- a/src/com/google/doclava/AnnotationInstanceInfo.java
+++ b/src/com/google/doclava/AnnotationInstanceInfo.java
@@ -120,27 +120,13 @@ public class AnnotationInstanceInfo implements Resolvable {
return allResolved;
}
- /**
- * Convert the specified list of {@code AnnotationInstanceInfo} into an HDF-formatted list, and
- * add the HDF list into the specified {@code Data}.
- */
public static void makeLinkListHDF(Data data, String base, AnnotationInstanceInfo[] annotations) {
if (annotations == null) return;
final int N = annotations.length;
for (int i = 0; i < N; i++) {
AnnotationInstanceInfo aii = annotations[i];
- final String aiiBase = base + "." + i;
-
- // Serialize data about the annotation element values
- for (int elemIdx = 0; elemIdx < aii.elementValues().size(); ++elemIdx) {
- final String elemBase = aiiBase + ".elementValues." + elemIdx;
- final AnnotationValueInfo value = aii.elementValues().get(elemIdx);
- data.setValue(elemBase + ".name", value.element().name());
- data.setValue(elemBase + ".value", value.valueString());
- }
-
- aii.type().makeShortDescrHDF(data, aiiBase);
+ aii.type().makeShortDescrHDF(data, base + "." + i);
}
}
diff --git a/src/com/google/doclava/MemberInfo.java b/src/com/google/doclava/MemberInfo.java
index da11360..76087eb 100644
--- a/src/com/google/doclava/MemberInfo.java
+++ b/src/com/google/doclava/MemberInfo.java
@@ -143,20 +143,9 @@ public abstract class MemberInfo extends DocInfo implements Comparable, Scoped {
return mContainingClass;
}
- /**
- * Returns {@code true} if the member's scope is above the minimum requested scope passed to
- * Doclava, <emph>or</emph> if the member is tagged with an annotation which was specified in a
- * "-showAnnotation" argument to Doclava
- */
public boolean checkLevel() {
- if (Doclava.checkLevel(mIsPublic, mIsProtected, mIsPackagePrivate, mIsPrivate,
- isHiddenOrRemoved())) {
- return true;
- } else if (mShowAnnotations != null && !mShowAnnotations.isEmpty()) {
- return true;
- }
-
- return false;
+ return Doclava.checkLevel(mIsPublic, mIsProtected, mIsPackagePrivate, mIsPrivate,
+ isHiddenOrRemoved());
}
public String kind() {