From f6822c973dd67656fe4be665a202b89238535041 Mon Sep 17 00:00:00 2001 From: cketti Date: Mon, 20 Oct 2014 21:13:58 -0400 Subject: [PATCH] Clean up comments --- src/com/fsck/k9/view/AttachmentView.java | 23 ++++------------------- 1 file changed, 4 insertions(+), 19 deletions(-) diff --git a/src/com/fsck/k9/view/AttachmentView.java b/src/com/fsck/k9/view/AttachmentView.java index 0dbec5bab..55a874544 100644 --- a/src/com/fsck/k9/view/AttachmentView.java +++ b/src/com/fsck/k9/view/AttachmentView.java @@ -98,16 +98,7 @@ public class AttachmentView extends FrameLayout implements OnClickListener, OnLo * Inline attachments with content ID and unnamed attachments fall into the second category. *

* - * @param inputPart - * @param message - * @param account - * @param controller - * @param listener - * - * @return {@code true} for a regular attachment. {@code false}, otherwise. - * - * @throws MessagingException - * In case of an error + * @return {@code true} for a regular attachment. {@code false} for attachments that should be initially hidden. */ public boolean populateFromPart(Part inputPart, Message message, Account account, MessagingController controller, MessagingListener listener) throws MessagingException { @@ -229,10 +220,7 @@ public class AttachmentView extends FrameLayout implements OnClickListener, OnLo } /** - * Writes the attachment onto the given path - * - * @param directory - * the base dir where the file should be saved. + * Saves the attachment as file in the given directory */ public void writeFile(File directory) { try { @@ -255,10 +243,6 @@ public class AttachmentView extends FrameLayout implements OnClickListener, OnLo } } - /** - * saves the file to the defaultpath setting in the config, or if the config - * is not set => to the Environment - */ public void writeFile() { writeFile(new File(K9.getAttachmentDefaultPath())); } @@ -283,7 +267,8 @@ public class AttachmentView extends FrameLayout implements OnClickListener, OnLo public void showFile() { Uri uri = AttachmentProvider.getAttachmentUriForViewing(account, part.getAttachmentId()); Intent intent = new Intent(Intent.ACTION_VIEW); - // We explicitly set the ContentType in addition to the URI because some attachment viewers (such as Polaris office 3.0.x) choke on documents without a mime type + // We explicitly set the content type in addition to the URI because some attachment viewers + // (such as Polaris office 3.0.x) choke on documents without a MIME type. intent.setDataAndType(uri, contentType); intent.addFlags(Intent.FLAG_GRANT_READ_URI_PERMISSION | Intent.FLAG_ACTIVITY_CLEAR_WHEN_TASK_RESET);