> On Nov. 22, 2014, 4:40 p.m., David Gil Oliva wrote:
> > src/kled.h, line 257
> > <https://git.reviewboard.kde.org/r/121210/diff/1/?file=329497#file329497line257>
> >
> >     It should be hidden behind the d-pointer.
> 
> José Millán Soto wrote:
>     Why should it be hidden?
>     As it is not a virtual function it should not cause binary 
> incompatibility.
> 
> David Gil Oliva wrote:
>     I tend to think That everything private should be behind the d-pointer. 
> It keeps a tighter encapsulation. But maybe I'm wrong...

Generally, I agree, but the Private class currently has no q-pointer back to 
the widget, and I do not think it is worth adding it just for this single 
function.


- Christoph


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/121210/#review70783
-----------------------------------------------------------


On Nov. 22, 2014, 4:32 p.m., José Millán Soto wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121210/
> -----------------------------------------------------------
> 
> (Updated Nov. 22, 2014, 4:32 p.m.)
> 
> 
> Review request for KDE Accessibility and KDE Frameworks.
> 
> 
> Repository: kwidgetsaddons
> 
> 
> Description
> -------
> 
> This patch will provide a default accessible name for KLed. That name will 
> only say that the widget is a led and the status of the led. If another 
> accessible name has been assigned, it will not be overwritten.
> The accessible name will allow accessible tools (such as screen readers) to 
> provide some infomation about the widget.
> 
> 
> Diffs
> -----
> 
>   src/kled.h eeb1209 
>   src/kled.cpp 9788fc2 
> 
> Diff: https://git.reviewboard.kde.org/r/121210/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> José Millán Soto
> 
>

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to