[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: koffice/plugins/chartshape
From:       Johannes Simon <johannes.simon () gmail ! com>
Date:       2010-10-22 19:15:06
Message-ID: A40C3A8B-7674-47ED-9CBC-00BBB23836A4 () gmail ! com
[Download RAW message or body]

Am 22.10.2010 um 14:37 schrieb Pavol Korinek:
> break;
> case LineChartType:
> @@ -1611,6 +1616,7 @@
> type = KDChart::LineDiagram::Stacked; break;
> case PercentChartSubtype:
> type = KDChart::LineDiagram::Percent; break;
> +                d->kdBarDiagram->setUnitSuffix("%", \
> d->kdBarDiagram->orientation());
I think this is somewhat pointless considering the "break;" before the newline ;-) \
Also didn't you mean d->kdLineDiagram here?
> default:
> type = KDChart::LineDiagram::Normal;
> }
> @@ -1625,6 +1631,7 @@
> type = KDChart::LineDiagram::Stacked; break;
> case PercentChartSubtype:
> type = KDChart::LineDiagram::Percent; break;
> +                d->kdBarDiagram->setUnitSuffix("%", \
> d->kdBarDiagram->orientation());
Same here. (shouldn't it be d->kdAreaDiagram?)
> default:
> type = KDChart::LineDiagram::Normal;
> }


[prev in list] [next in list] [prev in thread] [next in thread]