Review Request: Add "open-with" action and context-popmenu to ark

[Raphael Kubo da Costa]

> On Jan. 29, 2012, 2:08 a.m., Raphael Kubo da Costa wrote:
> > Thank you for working on this. There is still some work ahead, but I think the issues can be sorted out.
> > 
> > Besides the specific points I have raised below, a few general comments:
> >  - There are many coding style issues with the patch, mostly related to white space.
> >    - Trailing white space
> >    - foo( bar ) and foo[ bar ] being used instead of foo(bar) and foo[bar]. Ark follows kdelibs' and Qt's coding conventions.
> >  - Please add the "kdeutils" group to the review request.
> >  - I disagree with the "DefaultApplication" action and the way it is triggered. It is totally not clear to the user that the default behavior is now to try to launch the default application associated with the file's mimetype than to use the previewer. This is also unrelated to the main objective of the patch.
> >  - Trying to preview a file with the default application always crashes Ark here.
> >  - The version in ark_part.rc needs to be bumped.
> 
> Volker Härtel wrote:
>     Thanks for your review, I'll try to fix the issues soon. Can you give me a hint about the crashes on preview? I did not have any crashes.

Relevant part of the backtrace:

#0  0x000000080574d40c in thr_kill () from /lib/libc.so.7
#1  0x00000008057e84ab in abort () from /lib/libc.so.7
#2  0x0000000804aa4e7f in qt_message_output (msgType=QtFatalMsg, 
    buf=0x80bdd7818 "ASSERT: \"!kioexec.isEmpty()\" in file /home/rakuco/kde4/src/kde/kdelibs/kio/kio/krun.cpp, line 422") at /usr/home/rakuco/kde4/src/qt-4.8/src/corelib/global/qglobal.cpp:2252
#3  0x0000000804aa4ff2 in qt_message(QtMsgType, const char *, typedef __va_list_tag __va_list_tag *)
    (msgType=QtFatalMsg, msg=0x804c64960 "ASSERT: \"%s\" in file %s, line %d", ap=0x7fffffffac58)
    at /usr/home/rakuco/kde4/src/qt-4.8/src/corelib/global/qglobal.cpp:2298
#4  0x0000000804aa5782 in qFatal (msg=0x804c64960 "ASSERT: \"%s\" in file %s, line %d")
    at /usr/home/rakuco/kde4/src/qt-4.8/src/corelib/global/qglobal.cpp:2481
#5  0x0000000804aa4a3c in qt_assert (assertion=0x80117167f "!kioexec.isEmpty()", 
    file=0x801170c70 "/home/rakuco/kde4/src/kde/kdelibs/kio/kio/krun.cpp", line=422)
    at /usr/home/rakuco/kde4/src/qt-4.8/src/corelib/global/qglobal.cpp:1999
#6  0x0000000801088b44 in KRun::processDesktopExec (_service=..., _urls=..., tempFiles=true, 
    suggestedFileName=...) at /home/rakuco/kde4/src/kde/kdelibs/kio/kio/krun.cpp:422
#7  0x000000080108a2fa in runTempService (_service=..., _urls=..., window=0x80bd16f10, 
    tempFiles=true, suggestedFileName=..., asn=...)
    at /home/rakuco/kde4/src/kde/kdelibs/kio/kio/krun.cpp:709
#8  0x000000080108a8e8 in KRun::run (_service=..., _urls=..., window=0x80bd16f10, tempFiles=true, 
    suggestedFileName=..., asn=...) at /home/rakuco/kde4/src/kde/kdelibs/kio/kio/krun.cpp:988
#9  0x000000080108c9a4 in KRun::runUrl (u=..., _mimetype=..., window=0x80bd16f10, tempFile=true, 
    runExecutables=true, suggestedFileName=..., asn=...)
    at /home/rakuco/kde4/src/kde/kdelibs/kio/kio/krun.cpp:183
#10 0x000000080f622ef7 in Ark::Part::slotPreviewExtracted (this=0x80bc62100, job=<optimized out>)
    at /home/rakuco/kde4/src/kdeutils/master/ark/part/part.cpp:594


- Raphael


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


On Jan. 29, 2012, 9:04 a.m., Volker Härtel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103690/
> -----------------------------------------------------------
> 
> (Updated Jan. 29, 2012, 9:04 a.m.)
> 
> 
> Review request for KDE Utils and Raphael Kubo da Costa.
> 
> 
> Description
> -------
> 
> Add an "open with" action to ark. Also add a context-menu (popup) for typical actions to ark. Change standard way to open files to preferred application.
> 
> This patch is based on https://svn.reviewboard.kde.org/r/6580/
> 
> 
> This addresses bug 179066.
>     http://bugs.kde.org/show_bug.cgi?id=179066
> 
> 
> Diffs
> -----
> 
>   part/archiveview.h daac59b 
>   part/archiveview.cpp 6b9918d 
>   part/ark_part.rc 044c11a 
>   part/part.h 5379b9f 
>   part/part.cpp aa4756a 
> 
> Diff: http://git.reviewboard.kde.org/r/103690/diff/diff
> 
> 
> Testing
> -------
> 
> Works for me
> 
> 
> Thanks,
> 
> Volker Härtel
> 
>

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