[Kde-pim] Review Request: Krazy fixes and simple compiler warnings

Thu Mar 26 21:09:27 GMT 2009

> On 2009-03-22 15:07:23, Thomas McGuire wrote:
> > trunk/KDE/kdepim/akregator/interfaces/feedstorage.h, line 68
> > <http://reviewboard.kde.org/r/398/diff/1/?file=3776#file3776line68>
> >
> >     Why this change?

Removed in the commit.

> On 2009-03-22 15:07:23, Thomas McGuire wrote:
> > trunk/KDE/kdepim/akregator/plugins/onlinesync/onlinesyncplugin.cpp, line 48
> > <http://reviewboard.kde.org/r/398/diff/1/?file=3779#file3779line48>
> >
> >     coding style: Q_UNUSED( list )
> >     Also in other places

Adapted to the coding style of the file (this was mostly in akregator, not KMail).

- nik

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

On 2009-03-26 14:09:07, nik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/398/
> -----------------------------------------------------------
> 
> (Updated 2009-03-26 14:09:07)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> Krazy fixes and simple compiler warnings for kdepim/.
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdepim/akonadiconsole/agentwidget.cpp 945031 
>   trunk/KDE/kdepim/akonadiconsole/rawsocketconsole.cpp 945031 
>   trunk/KDE/kdepim/akregator/configuration/akregator_config_appearance.h 945031 
>   trunk/KDE/kdepim/akregator/configuration/akregator_config_archive.h 945031 
>   trunk/KDE/kdepim/akregator/configuration/akregator_config_browser.h 945031 
>   trunk/KDE/kdepim/akregator/configuration/akregator_config_general.h 945031 
>   trunk/KDE/kdepim/akregator/interfaces/storagefactoryregistry.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/mk4storage/feedstoragemk4impl.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/akregator_config_onlinesync.h 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/onlinesyncplugin.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/sync/akregator.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/sync/feedsync.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/sync/googlereader.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/sync/opml.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/sync/subscriptionlist.cpp 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/ui/configurationdialog.h 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/ui/configurationdialogadd.h 945031 
>   trunk/KDE/kdepim/akregator/plugins/onlinesync/ui/configurationwidget.h 945031 
>   trunk/KDE/kdepim/akregator/src/addfeeddialog.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/akregator_part.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/article.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/articleformatter.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/articleviewer.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/browserframe.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/feed.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/feedlist.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/feedpropertiesdialog.h 945031 
>   trunk/KDE/kdepim/akregator/src/feedpropertiesdialog.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/frame.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/framemanager.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/mainwidget.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/notificationmanager.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/pageviewer.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/pluginmanager.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/progressmanager.h 945031 
>   trunk/KDE/kdepim/akregator/src/progressmanager.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/searchbar.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/speechclient.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/subscriptionlistmodel.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/subscriptionlistview.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/treenode.cpp 945031 
>   trunk/KDE/kdepim/akregator/src/treenodevisitor.cpp 945031 
>   trunk/KDE/kdepim/kaddressbook/views/configuretableviewdialog.cpp 945031 
>   trunk/KDE/kdepim/kaddressbook/xxport/csvimportdialog.cpp 945031 
>   trunk/KDE/kdepim/kaddressbook/xxport/eudora_xxport.cpp 945031 
>   trunk/KDE/kdepim/kaddressbook/xxport/gmx_xxport.cpp 945031 
>   trunk/KDE/kdepim/kaddressbook/xxport/gnokii_xxport.cpp 945031 
>   trunk/KDE/kdepim/kalarm/resources/resourcelocaldir.cpp 945031 
>   trunk/KDE/kdepim/kjots/bookshelf.cpp 945031 
>   trunk/KDE/kdepim/kjots/kjotscomponent.h 945031 
>   trunk/KDE/kdepim/kjots/kjotscomponent.cpp 945031 
>   trunk/KDE/kdepim/kjots/kjotsentry.cpp 945031 
>   trunk/KDE/kdepim/kjots/knowitimporter.cpp 945031 
>   trunk/KDE/kdepim/kmail/kmmessage.cpp 945031 
>   trunk/KDE/kdepim/kmail/messagecomposer.cpp 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_evolution_v2.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_kmail_maildir.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_mailapp.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_oe.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_pmail.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_sylpheed.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/filter_thunderbird.cxx 945031 
>   trunk/KDE/kdepim/kmailcvt/kimportpage.cpp 945031 
>   trunk/KDE/kdepim/kmailcvt/kselfilterpage.cpp 945031 
>   trunk/KDE/kdepim/knotes/knote.h 945031 
>   trunk/KDE/kdepim/knotes/knote.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotealarmdlg.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotebutton.cpp 945031 
>   trunk/KDE/kdepim/knotes/knoteconfigdlg.cpp 945031 
>   trunk/KDE/kdepim/knotes/knoteedit.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotehostdlg.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotesalarm.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotesapp.cpp 945031 
>   trunk/KDE/kdepim/knotes/knoteslegacy.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotesnetrecv.h 945031 
>   trunk/KDE/kdepim/knotes/knotesnetrecv.cpp 945031 
>   trunk/KDE/kdepim/knotes/knotesnetsend.cpp 945031 
>   trunk/KDE/kdepim/knotes/resourcelocal.cpp 945031 
>   trunk/KDE/kdepim/knotes/resourcelocalconfig.cpp 945031 
>   trunk/KDE/kdepim/knotes/resourcemanager.cpp 945031 
>   trunk/KDE/kdepim/korganizer/views/agendaview/koagendaitem.h 945031 
>   trunk/KDE/kdepim/kpilot/conduits/base/recordconduit.h 945031 
>   trunk/KDE/kdepim/kpilot/conduits/calendar/calendarconduit.h 945031 
>   trunk/KDE/kdepim/kpilot/conduits/contacts/contactsconduit.h 945031 
>   trunk/KDE/kdepim/kpilot/conduits/todo/todoconduit.h 945031 
>   trunk/KDE/kdepim/ksendemail/mailerservice.cpp 945031 
>   trunk/KDE/kdepim/ktimetracker/karmstorage.h 945031 
>   trunk/KDE/kdepim/ktimetracker/karmstorage.cpp 945031 
>   trunk/KDE/kdepim/ktimetracker/taskview.cpp 945031 
>   trunk/KDE/kdepim/libksieve/impl/parser.h 945031 
>   trunk/KDE/kdepim/libksieve/parser/lexer.cpp 945031 
>   trunk/KDE/kdepim/plugins/kmail/bodypartformatter/text_xdiff.cpp 945031 
>   trunk/KDE/kdepim/strigi-analyzer/ics/icsendanalyzer.h 945031 
>   trunk/KDE/kdepim/strigi-analyzer/ics/icsendanalyzer.cpp 945031 
>   trunk/KDE/kdepim/strigi-analyzer/vcf/vcfendanalyzer.h 945031 
>   trunk/KDE/kdepim/strigi-analyzer/vcf/vcfendanalyzer.cpp 945031 
>   trunk/KDE/kdepim/wizards/egroupwarewizard.cpp 945031 
>   trunk/KDE/kdepim/wizards/kolabwizard.cpp 945031 
> 
> Diff: http://reviewboard.kde.org/r/398/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> nik
> 
>

_______________________________________________
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/