[Kde-pim] Review Request 109198: when removing maildir folder X, also remove .X.directory with subfolders

[Andras Mantia]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109198/#review28432
-----------------------------------------------------------

Ship it!


Looks good, you can also change the QString::fromLatin1( ".%1.directory" ).arg( folderName ) codes to subDirNameForFolderName( const QString &folderName ). Please commit to 4.10 branch and merge the branch to master after.

- Andras Mantia


On Feb. 28, 2013, 12:12 a.m., Wolfgang Rohdewald wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109198/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2013, 12:12 a.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> until now, the contents of the subfolders in .X.directory have already been deleted but the directory structure remained
> 
> there are more places in maildir.cpp where subDirNameForFolderName could be used instead of QString(".%1.directory").arg(folderName), 
> but maybe I can simply do that without review?
> 
> 
> Diffs
> -----
> 
>   resources/maildir/libmaildir/maildir.cpp 6355b4ab44ecada94a379b82c52e1315a2894eb7 
> 
> Diff: http://git.reviewboard.kde.org/r/109198/diff/
> 
> 
> Testing
> -------
> 
> removing maildir folders with subfolders (using kmail), readding them
> 
> 
> Thanks,
> 
> Wolfgang Rohdewald
> 
>

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/