----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104797/#review13520 -----------------------------------------------------------
libs/koreport/renderer/KoReportRendererBase.cpp <http://git.reviewboard.kde.org/r/104797/#comment10684> Since it's a new code, I propose to get the habit of using QString::compare(..., ..., Qt::CaseInsensitive) libs/koreport/renderer/odt/KoSimpleOdtCheckBox.cpp <http://git.reviewboard.kde.org/r/104797/#comment10685> Is't 0.0pt enough? libs/koreport/renderer/odt/KoSimpleOdtCheckBox.cpp <http://git.reviewboard.kde.org/r/104797/#comment10691> isn't 0.0pt better? libs/koreport/renderer/odt/KoSimpleOdtDocument.cpp <http://git.reviewboard.kde.org/r/104797/#comment10686> how about using kWarning()? there are more such cases... libs/koreport/renderer/odt/KoSimpleOdtLine.cpp <http://git.reviewboard.kde.org/r/104797/#comment10687> isn't 0.0pt better? libs/koreport/renderer/odt/KoSimpleOdtPrimitive.cpp <http://git.reviewboard.kde.org/r/104797/#comment10688> Let's keep such notes as @note in the header's doxygen comments for this method. libs/koreport/renderer/odt/KoSimpleOdtPrimitive.cpp <http://git.reviewboard.kde.org/r/104797/#comment10689> isn't 0.0pt better? libs/koreport/renderer/odt/KoSimpleOdtPrimitive.cpp <http://git.reviewboard.kde.org/r/104797/#comment10690> isn't 0.0pt better? - Jarosław Staniek On April 30, 2012, 4:31 p.m., Dag Andersen wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104797/ > ----------------------------------------------------------- > > (Updated April 30, 2012, 4:31 p.m.) > > > Review request for Calligra, Jarosław Staniek and Adam Pigg. > > > Description > ------- > > Changes is in branch: report_odt_danders > > Move plans odt export into libs/koreport. > > Retains the current odt export in koreport (as odttables) and > adds the new as odtframes. > > Patch includes modifications to Plan to enable use of both formats. > > > Diffs > ----- > > libs/koreport/CMakeLists.txt d0737c9 > libs/koreport/renderer/KoReportODTRenderer2.h PRE-CREATION > libs/koreport/renderer/KoReportODTRenderer2.cpp PRE-CREATION > libs/koreport/renderer/KoReportRendererBase.cpp 497177d > libs/koreport/renderer/odt/KoSimpleOdtCheckBox.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtCheckBox.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtDocument.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtDocument.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtImage.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtImage.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtLine.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtLine.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPicture.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPicture.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPrimitive.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtPrimitive.cpp PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtTextBox.h PRE-CREATION > libs/koreport/renderer/odt/KoSimpleOdtTextBox.cpp PRE-CREATION > libs/koreport/renderer/odt/ReportODTRenderer.h PRE-CREATION > plan/libs/ui/CMakeLists.txt 4948bd3 > plan/libs/ui/reports/odt/KoSimpleOdtCheckBox.h 94123e2 > plan/libs/ui/reports/odt/KoSimpleOdtCheckBox.cpp 9a94f5b > plan/libs/ui/reports/odt/KoSimpleOdtDocument.h 65bbbee > plan/libs/ui/reports/odt/KoSimpleOdtDocument.cpp 7d917ef > plan/libs/ui/reports/odt/KoSimpleOdtImage.h 3c40922 > plan/libs/ui/reports/odt/KoSimpleOdtImage.cpp 1061664 > plan/libs/ui/reports/odt/KoSimpleOdtLine.h 6ea097b > plan/libs/ui/reports/odt/KoSimpleOdtLine.cpp 2347117 > plan/libs/ui/reports/odt/KoSimpleOdtPicture.h 10c3b85 > plan/libs/ui/reports/odt/KoSimpleOdtPicture.cpp e0df27f > plan/libs/ui/reports/odt/KoSimpleOdtPrimitive.h 7cff1c7 > plan/libs/ui/reports/odt/KoSimpleOdtPrimitive.cpp ddea498 > plan/libs/ui/reports/odt/KoSimpleOdtTextBox.h 428dff7 > plan/libs/ui/reports/odt/KoSimpleOdtTextBox.cpp d775689 > plan/libs/ui/reports/odt/ReportODTRenderer.h 3c9f149 > plan/libs/ui/reports/odt/ReportODTRenderer.cpp 2c95ed2 > plan/libs/ui/reports/report.h 8cef5be > plan/libs/ui/reports/reportexportpanel.cpp 973078d > plan/libs/ui/reports/reportview.h 265080a > plan/libs/ui/reports/reportview.cpp dea21a6 > > Diff: http://git.reviewboard.kde.org/r/104797/diff/ > > > Testing > ------- > > Tested with Plan, resulting files opened with Words. > > > Thanks, > > Dag Andersen > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel