Review Request: Add option to qalculate to show the result in different bases at the same time
Jef Steelant
jef at steelant.be
Thu Feb 3 16:59:03 CET 2011
> On Feb. 3, 2011, 3:15 p.m., Matteo Agostinelli wrote:
> > applets/qalculate/qalculate_labels.h, lines 32-56
> > <http://git.reviewboard.kde.org/r/100533/diff/1/?file=8207#file8207line32>
> >
> > I don't understand why you created the AppletSettings class. Wouldn't it be easier to simply pass a pointer to QalculateSettings in the constructor?
> >
> > This would save some code duplication. Or am I missing something?
I did not want to drag the whole QalculateSettings around, but it is probably a better idea anyway. I'll update the code first.
- Jef
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100533/#review1178
-----------------------------------------------------------
On Feb. 3, 2011, 3:58 p.m., Jef Steelant wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100533/
> -----------------------------------------------------------
>
> (Updated Feb. 3, 2011, 3:58 p.m.)
>
>
> Review request for Plasma.
>
>
> Summary
> -------
>
> Add the option to qalculate to show the result in binary, octal, decimal and hexadecimal at the same time.
> This only works if the result is an integer
>
>
> Diffs
> -----
>
> applets/qalculate/qalculate_engine.cpp 708f3b2
> applets/qalculate/qalculate_labels.h 4cf31e2
> applets/qalculate/qalculate_labels.cpp b04c538
> applets/qalculate/qalculate_settings.h cab44e9
> applets/qalculate/qalculate_settings.cpp b317e67
>
> Diff: http://git.reviewboard.kde.org/r/100533/diff
>
>
> Testing
> -------
>
> built and tested the applet
>
>
> Screenshots
> -----------
>
> settings
> http://git.reviewboard.kde.org/r/100533/s/63/
> result
> http://git.reviewboard.kde.org/r/100533/s/64/
>
>
> Thanks,
>
> Jef
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110203/5572a35c/attachment-0001.htm
More information about the Plasma-devel
mailing list