Review Request 115324: Remove extra include from taglib to allow older versions of taglib to be used.

Commit Hook null at kde.org
Mon Jan 27 05:13:08 UTC 2014


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


This review has been submitted with commit b7eff183a5a79ee02bf9f183ec2393f42486c3ca by Matthew Dawson to branch master.

- Commit Hook


On Jan. 27, 2014, 4:27 a.m., Matthew John Dawson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115324/
> -----------------------------------------------------------
> 
> (Updated Jan. 27, 2014, 4:27 a.m.)
> 
> 
> Review request for Plasma, Shantanu Tushar and Sinny Kumari.
> 
> 
> Repository: plasma-mediacenter
> 
> 
> Description
> -------
> 
> Plasma Mediacenter includes an extra file from taglib that only exists in taglib 1.8.  However, this include is not actually used, and not using it allows Plasma Mediacenter to compile with taglib 1.7.  This change is required for Plasma Active as it is using taglib 1.7 for now.
> 
> 
> Diffs
> -----
> 
>   libs/mediacenter/mediainfoservice.cpp fb52b03 
> 
> Diff: https://git.reviewboard.kde.org/r/115324/diff/
> 
> 
> Testing
> -------
> 
> Compiling with both gcc and clang produce no warnings or errors.  gcc generates the exact same assembly for both versions.
> 
> 
> Thanks,
> 
> Matthew John Dawson
> 
>

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


More information about the Plasma-devel mailing list