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