* docu fixes

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.
 

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;
 		}

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;

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);