> On July 12, 2012, 10:15 p.m., C. Boemann wrote: > > I could have sworn I saw a ship-it by Dag for this, or was that a similar > > request for Plan? > > > > > > Friedrich W. H. Kossebau wrote: > Yes, I splitted between Sheets and Plan, hoping to split the review > effort needed. > But currently most of the reviews seem to end up with you anyway :) > > Many thanks for all the reviewing, really great to see my requests not > catching any dust (unless quite tricky or big, like the koIcon one)! > > I feel bad not being responsible for anything myself, so I do not have to > do reviewing myself :) But my interest currently simply jumps around between > all places in Calligra (well, to the issues I see).
Heh, Well I do feel a bit stressed by all your requests. I wish more people would be able to help reviewing, or you would slow down. Well I don't really want you to slow down. You are doing a great job, many thanks, but it does takes it's toll :) And you don't have to be responsible of anything to do a review. It's not supposed to be a maintainer's approval, but just peer review, so if you see some requests where you know something or can just check for basic saneness or run a compile or whatever seems appropriate reviewing, by all means do it. As for this request, since the similar one have already been approved and worked without problems, let's approve this one too - C. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105501/#review15752 ----------------------------------------------------------- On July 10, 2012, 6:18 a.m., Friedrich W. H. Kossebau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105501/ > ----------------------------------------------------------- > > (Updated July 10, 2012, 6:18 a.m.) > > > Review request for Calligra and Marijn Kruisselbrink. > > > Description > ------- > > In kdelibs 4.4 KLocale::fragDigits was split into > KLocale::monetaryDecimalNumbers and KLocale::decimalNumbers. > > While the deprecated KLocale::fragDigits at least in 4.8 maps to > KLocale::monetaryDecimalNumbers (though not documented in the API, just > implementation), for me it still feels better to use the correct property > directly if compiled against 4.4 and later. And in > OpenCalcExport::exportDefaultCellStyle() it should be rather > KLocale::decimalNumbers, not KLocale::monetaryDecimalNumbers, from what I > understand. > Also cleans the build output from the "Deprecated!" warnings :) > > Patch also fixes Localization::defaultSystemConfig() still using "tables" and > not "sheets" as id. > > > Diffs > ----- > > filters/sheets/opencalc/opencalcexport.cc 383d328 > sheets/Localization.cpp 6989f4e > sheets/ui/CellToolBase.cpp 89b79f9 > sheets/ui/CellToolBase_p.cpp bbfe480 > > Diff: http://git.reviewboard.kde.org/r/105501/diff/ > > > Testing > ------- > > > Thanks, > > Friedrich W. H. Kossebau > >
_______________________________________________ calligra-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/calligra-devel
