Review Request 127498: Cleanup warnings

[Sebastian Kügler]

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


Fix it, then Ship it!




Ship It!


src/card.cpp (line 47)
<https://git.reviewboard.kde.org/r/127498/#comment64076>

    Perhaps we can just drop this conditional and make it require PA >= 5.0?
    
    The reason is to reduce silent failures, #ifdefs sounds like a neat thing, but it quickly gets out of hand when people expect software to have the same features, and then it doesn't, but they don't see build failures.
    
    My (current) Kubuntu has 6.0, my Debian Unstable has 8.0, I haven't investigated others, but it doesn't seem outrageous to require PA 5.0 for Plasma 5.7?


- Sebastian Kügler


On March 26, 2016, 9:07 a.m., David Rosca wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127498/
> -----------------------------------------------------------
> 
> (Updated March 26, 2016, 9:07 a.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-pa
> 
> 
> Description
> -------
> 
> Change some warnings to todo/fixme and fix the others.
> 
> 
> Diffs
> -----
> 
>   src/card.cpp 2e251a7 
>   src/context.cpp 115c8a7 
>   src/pulseaudio.cpp 1488784 
> 
> Diff: https://git.reviewboard.kde.org/r/127498/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Rosca
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160331/116ba906/attachment-0001.html>