Review Request 128633: Wether nor not install the private header?

Sune Vuorela kde at pusling.com
Tue Aug 9 06:20:04 UTC 2016


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



I think it was a bug that the individual barcode headers was public in the qt4 based prison. Now that we are about to release a qt5 based prison, it is time to fix that bug.

Basically, I don't want the interface to change depending on what supposed backends prison is compiled with.

createBarcode can be queried, and the result tested for null pointers if a specific feature is not compiled in. At runtime. So things can be enabled/disabled as needed without requiring changes in other parts of the code.

It is a very deliberate action to not install the headers any more.

- Sune Vuorela


On Aug. 9, 2016, 6:03 a.m., Leslie Zhai wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128633/
> -----------------------------------------------------------
> 
> (Updated Aug. 9, 2016, 6:03 a.m.)
> 
> 
> Review request for Plasma and Sune Vuorela.
> 
> 
> Repository: prison
> 
> 
> Description
> -------
> 
> Hi Sune,
> 
> People might directly use Prison::QRCodeBarcode https://github.com/xiangzhai/qwx/blob/kf5/src/qrcodequick.cpp#L9 instead of via Prison::createBarcode
> 
> so is it able to install the private header?
> 
> Regards,
> Leslie Zhai
> 
> 
> Diffs
> -----
> 
>   lib/CMakeLists.txt 95391cc 
> 
> Diff: https://git.reviewboard.kde.org/r/128633/diff/
> 
> 
> Testing
> -------
> 
> - prison/tests/test-prison
> - plasma-workspace/klipper
> - qwx/QRcodeQuick https://github.com/xiangzhai/qwx/blob/kf5/src/qrcodequick.h
> 
> 
> Thanks,
> 
> Leslie Zhai
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160809/e3c14409/attachment.html>


More information about the Plasma-devel mailing list