From f7c2ee5f49d1b261eb6025f6fdba24df2569621f Mon Sep 17 00:00:00 2001
From: "Dirk-Jan C. Binnema" <djcb@djcbsoftware.nl>
Date: Mon, 8 Aug 2011 21:57:48 +0300
Subject: [PATCH] * docu fixes

---
 man/mu-mv.1        |  2 +-
 src/mu-cmd.c       |  3 ++-
 src/mu-msg-flags.h | 10 +++++++---
 src/mu-msg.c       |  2 ++
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/man/mu-mv.1 b/man/mu-mv.1
index e51b3808..ebb7bedd 100644
--- a/man/mu-mv.1
+++ b/man/mu-mv.1
@@ -41,7 +41,7 @@ using the this option, you can change the file flags of the target file. If
 you change the 'N' (new) flag, this will also change the exact target
 directory ('new' vs 'cur').
 
-The flags is a sequence of characters from the set D (draft), F (flagged) ,N
+The flags is a sequence of characters from the set D (draft), F (flagged), N
 (new), P (passed), R (replied), S (seen) and T (trashed). Note, the
 flags-parameter is case-sensitive.
 
diff --git a/src/mu-cmd.c b/src/mu-cmd.c
index c9c77cdd..b12147fd 100644
--- a/src/mu-cmd.c
+++ b/src/mu-cmd.c
@@ -317,7 +317,8 @@ mu_cmd_mv (MuConfig *opts)
 			g_warning ("unlink failed: %s", strerror (errno));
 			return MU_EXITCODE_ERROR;
 		} else {
-			if (opts->printtarget)                 /* if the move worked, print */
+			if (opts->printtarget)
+				/* if the move worked, print */
 				g_print ("%s\n", "/dev/null"); /* /dev/null */
 			return MU_EXITCODE_OK;
 		}
diff --git a/src/mu-msg-flags.h b/src/mu-msg-flags.h
index c2fce867..4a2f83f8 100644
--- a/src/mu-msg-flags.h
+++ b/src/mu-msg-flags.h
@@ -56,17 +56,21 @@ enum _MuMsgFlags {
 	/* "F"->flagged message */
 	MU_MSG_FLAG_FLAGGED	= 1 << 6,
 
+	/* NOTE: the only flag characters actually seen in Maildir
+	 * files are: PRSTDF
+	 */
+	
 	/* "U"->unread message; it's a pseudo/convenience flag that
 	 * means (NEW or not SEEN) */
 	MU_MSG_FLAG_UNREAD      = 1 << 7,
 	
 	/* these we get from the contents */
 	
-	/* "Z"->signed message */
+	/* "z"->signed message */
 	MU_MSG_FLAG_SIGNED      = 1 << 8,
-	/* "X"->encrypted message */
+	/* "x"->encrypted message */
 	MU_MSG_FLAG_ENCRYPTED   = 1 << 9,
-	/* "A"->message has attachment */
+	/* "a"->message has attachment */
 	MU_MSG_FLAG_HAS_ATTACH  = 1 << 10
 };
 typedef enum _MuMsgFlags MuMsgFlags;
diff --git a/src/mu-msg.c b/src/mu-msg.c
index 0b1ac9e6..5522a641 100644
--- a/src/mu-msg.c
+++ b/src/mu-msg.c
@@ -844,6 +844,8 @@ mu_msg_file_move_to_maildir (const char* oldpath, const char* targetmdir,
 			     "failed to determine target full path");
 		return FALSE;
 	}
+
+	/* TODO: check for oldpath == newfullpath */
 	
 	rv = msg_move (oldpath, newfullpath, err);