[k3b] Re: Review Request: Patch for windows build

[Michal Malek]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/100457/#review1147
-----------------------------------------------------------



libk3b/projects/audiocd/k3baudiocdtrackreader.cpp
<http://git.reviewboard.kde.org/r/100457/#comment958>

    I don't understand this one. What do you want to achieve here?
    Besides, it doesn't compile on GCC.



libk3bdevice/k3bdevice_mmc.cpp
<http://git.reviewboard.kde.org/r/100457/#comment959>

    Why do you want to allocate this dynamically? You create a memory leak here.


- Michal


On Jan. 29, 2011, 12:37 a.m., Andrius da Costa Ribas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/100457/
> -----------------------------------------------------------
> 
> (Updated Jan. 29, 2011, 12:37 a.m.)
> 
> 
> Review request for K3b, Michal Malek and Patrick Spendrin.
> 
> 
> Summary
> -------
> 
> Small patch to fix building on Windows
> 
> 
> Diffs
> -----
> 
>   cmake/modules/FindSamplerate.cmake 9afd66b 
>   libk3b/projects/audiocd/k3baudiocdtrackreader.cpp 83b6a43 
>   libk3b/tools/qprocess/qringbuffer_p.h 4b5ea48 
>   libk3bdevice/k3bdevice_mmc.cpp 4aa8dbf 
>   src/CMakeLists.txt fd630e4 
>   src/k3bemptydiscwaiter.cpp 738f63c 
>   src/projects/k3bencodingconverter.cpp 93d86f5 
> 
> Diff: http://git.reviewboard.kde.org/r/100457/diff
> 
> 
> Testing
> -------
> 
> Tested using MSVC 2008 on a Windows 7 machine
> 
> 
> Thanks,
> 
> Andrius da Costa
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/k3b/attachments/20110201/6163fa76/attachment.html>