----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105501/#review15761 -----------------------------------------------------------
Ship it! Ship It! - C. Boemann 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 calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
