Review Request 126341: app notifier is crashing right now. build a test around it

Aleix Pol Gonzalez aleixpol at kde.org
Mon Dec 14 10:42:01 UTC 2015


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

Ship it!


Ship It!

- Aleix Pol Gonzalez


On Dec. 14, 2015, 10:36 a.m., Harald Sitter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126341/
> -----------------------------------------------------------
> 
> (Updated Dec. 14, 2015, 10:36 a.m.)
> 
> 
> Review request for Kubuntu, Muon Package Management Suite and Aleix Pol Gonzalez.
> 
> 
> Repository: discover
> 
> 
> Description
> -------
> 
> prevent notifier from crashing when process was not initialized yet
> 
> 
> Diffs
> -----
> 
>   libdiscover/backends/ApplicationBackend/ApplicationNotifier.cpp 7611d046b5761b75f9535415b9b8903fef7ed49b 
>   libdiscover/backends/ApplicationBackend/CMakeLists.txt e2bb7fbeecde6983aa48b6bdb79a6c6fd41c83e3 
>   libdiscover/backends/ApplicationBackend/tests/CMakeLists.txt 274adf3a1f82ee8cd2f0f1bd6392d005b09195c6 
>   libdiscover/backends/ApplicationBackend/tests/NotifierTest.h PRE-CREATION 
>   libdiscover/backends/ApplicationBackend/tests/NotifierTest.cpp PRE-CREATION 
> 
> Diff: https://git.reviewboard.kde.org/r/126341/diff/
> 
> 
> Testing
> -------
> 
> ran test without fix -> exception fail
> ran test with fix -> pass
> 
> 
> Thanks,
> 
> Harald Sitter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kubuntu-devel/attachments/20151214/a1ca52f9/attachment.html>


More information about the kubuntu-devel mailing list