Review Request 126360: Ark 15.12 can't open rar file with space in path correctly.

[Elvis Angelaccio]

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



plugins/clirarplugin/cliplugin.cpp (line 296)
<https://git.reviewboard.kde.org/r/126360/#comment61283>

    Please apply the same fix also to unrar v4 parsing.


- Elvis Angelaccio


On Dec. 15, 2015, 8:37 a.m., Xuetian Weng wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/126360/
> -----------------------------------------------------------
> 
> (Updated Dec. 15, 2015, 8:37 a.m.)
> 
> 
> Review request for KDE Utils and Elvis Angelaccio.
> 
> 
> Repository: ark
> 
> 
> Description
> -------
> 
> The "Archive: " line can obviously contain space character, current code will take all output from unrar as comment if archive file path contains space.
> 
> This patch is only a naive change to capture all character on the Archive: line.
> 
> It seems it's possible that Comment section also contains "Archive:". So it doesn't fix all the problem. Probably parse from end would make it more robust.
> 
> 
> Diffs
> -----
> 
>   plugins/clirarplugin/cliplugin.cpp 23eb6cc 
> 
> Diff: https://git.reviewboard.kde.org/r/126360/diff/
> 
> 
> Testing
> -------
> 
> Open file with space in path, correctly file content is shown correctly.
> 
> 
> Thanks,
> 
> Xuetian Weng
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20151215/3e1786af/attachment.html>