<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/118293/">https://git.reviewboard.kde.org/r/118293/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">2) Generate a PDF document with latex, open it, change the source and regenerate it; wait until okular loads it, try to open the info dialog, crash:
Application: Okular (okular), signal: Segmentation fault
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
[KCrash Handler]
#5 ref (this=0x200000069) at /usr/include/qt4/QtCore/qatomic_x86_64.h:121
#6 operator= (other=..., this=0x175db40) at /usr/include/qt4/QtCore/qmap.h:433
#7 Okular::DocumentInfo::DocumentInfo (this=0x7fff4c5544f0, info=...) at /home/prova/kde-svn/git.kde.org/kde/kdegraphics/okular/core/document.cpp:4662
#8 0x00007ff4936e5871 in Okular::Document::documentInfo (this=this@entry=0x11a8f80, keys=...) at /home/prova/kde-svn/git.kde.org/kde/kdegraphics/okular/core/document.cpp:2587
#9 0x00007ff4936e63a0 in Okular::Document::documentInfo (this=0x11a8f80) at /home/prova/kde-svn/git.kde.org/kde/kdegraphics/okular/core/document.cpp:2582
#10 0x00007ff493a2b4eb in PropertiesDialog::PropertiesDialog (this=0x14f2fe0, parent=0x110, doc=0x16c3ea0) at /home/prova/kde-svn/git.kde.org/kde/kdegraphics/okular/ui/propertiesdialog.cpp:54
#11 0x00007ff4939aca3c in Okular::Part::slotShowProperties (this=0x11962c0) at /home/prova/kde-svn/git.kde.org/kde/kdegraphics/okular/part.cpp:2465
#12 0x00007ff4939b962d in Okular::Part::qt_static_metacall (_o=0x7ff4a1b55620 <main_arena>, _id=272, _a=0x16c3ea0, _c=<optimized out>) at /opt/kdebuild/kde/kdegraphics/okular/part.moc:232
#13 0x00007ff4a29a66aa in QMetaObject::activate(QObject*, QMetaObject const*, int, void**) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
#14 0x00007ff4a38d5572 in QAction::triggered(bool) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#15 0x00007ff4a38d6f43 in QAction::activate(QAction::ActionEvent) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#16 0x00007ff4a3d01959 in ?? () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#17 0x00007ff4a3d05ea9 in ?? () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#18 0x00007ff4a46b51f5 in KMenu::mouseReleaseEvent (this=0x1417340, e=0x7fff4c554ea0) at /home/prova/kde-svn/git.kde.org/kde/kdelibs/kdeui/widgets/kmenu.cpp:464
#19 0x00007ff4a392acca in QWidget::event(QEvent*) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#20 0x00007ff4a3d09d0b in QMenu::event(QEvent*) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#21 0x00007ff4a38db6cc in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#22 0x00007ff4a38e1e3a in QApplication::notify(QObject*, QEvent*) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#23 0x00007ff4a45fe83a in KApplication::notify (this=0x7fff4c555720, receiver=0x1417340, event=0x7fff4c554ea0) at /home/prova/kde-svn/git.kde.org/kde/kdelibs/kdeui/kernel/kapplication.cpp:311
#24 0x00007ff4a299241d in QCoreApplication::notifyInternal(QObject*, QEvent*) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
#25 0x00007ff4a38e1633 in QApplicationPrivate::sendMouseEvent(QWidget*, QMouseEvent*, QWidget*, QWidget*, QWidget**, QPointer<QWidget>&, bool) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#26 0x00007ff4a395366c in ?? () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#27 0x00007ff4a3951dac in QApplication::x11ProcessEvent(_XEvent*) () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#28 0x00007ff4a3979712 in ?? () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#29 0x00007ff49de5de04 in g_main_context_dispatch () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#30 0x00007ff49de5e048 in ?? () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#31 0x00007ff49de5e0ec in g_main_context_iteration () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
#32 0x00007ff4a29bf655 in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
#33 0x00007ff4a39797c6 in ?? () from /usr/lib/x86_64-linux-gnu/libQtGui.so.4
#34 0x00007ff4a2990fef in QEventLoop::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
#35 0x00007ff4a29912e5 in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
#36 0x00007ff4a2996ab9 in QCoreApplication::exec() () from /usr/lib/x86_64-linux-gnu/libQtCore.so.4
#37 0x0000000000409436 in main (argc=<optimized out>, argv=<optimized out>) at /home/prova/kde-svn/git.kde.org/kde/kdegraphics/okular/shell/main.cpp:174
</pre>
<br />
<p>- Luigi Toscano</p>
<br />
<p>On May 24th, 2014, 4:03 p.m. CEST, Albert Astals Cid wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Okular.</div>
<div>By Albert Astals Cid.</div>
<p style="color: grey;"><i>Updated May 24, 2014, 4:03 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
okular
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">DocumentInfo is now QMap based instead of XML based. Makes internal handling easier. The generators don't do any caching anymore since the Document was already doing caching, made no sense. Now we only ask for the keys we need (except the properties dialog) so the generators that are slow can decide to return only what was asked for. Some generators (i.e. unpluck, xps or the text based ones) will still return all the info since it's calculated on load time and it's not time consuming to return everything if we already have it.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Opened a few files, works as it did. Added some extra debug, made sure that only the info we want is asked for, and that it's properly cached.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>core/document.h <span style="color: grey">(06bac8a)</span></li>
<li>core/document.cpp <span style="color: grey">(504d07c)</span></li>
<li>core/document_p.h <span style="color: grey">(aabd192)</span></li>
<li>core/generator.h <span style="color: grey">(506f8a8)</span></li>
<li>core/generator.cpp <span style="color: grey">(ec1d1f7)</span></li>
<li>core/script/kjs_document.cpp <span style="color: grey">(c3280cd)</span></li>
<li>core/textdocumentgenerator.h <span style="color: grey">(ec331d6)</span></li>
<li>core/textdocumentgenerator.cpp <span style="color: grey">(d598ccc)</span></li>
<li>generators/chm/generator_chm.h <span style="color: grey">(5d97009)</span></li>
<li>generators/chm/generator_chm.cpp <span style="color: grey">(84b9dde)</span></li>
<li>generators/djvu/generator_djvu.h <span style="color: grey">(93ee4de)</span></li>
<li>generators/djvu/generator_djvu.cpp <span style="color: grey">(c506ce3)</span></li>
<li>generators/dvi/generator_dvi.h <span style="color: grey">(d7bf8fd)</span></li>
<li>generators/dvi/generator_dvi.cpp <span style="color: grey">(0777af7)</span></li>
<li>generators/fax/generator_fax.h <span style="color: grey">(1cb7d22)</span></li>
<li>generators/fax/generator_fax.cpp <span style="color: grey">(38873ef)</span></li>
<li>generators/kimgio/generator_kimgio.h <span style="color: grey">(faebd85)</span></li>
<li>generators/plucker/generator_plucker.h <span style="color: grey">(8dac4b8)</span></li>
<li>generators/plucker/generator_plucker.cpp <span style="color: grey">(4ffdead)</span></li>
<li>generators/poppler/generator_pdf.h <span style="color: grey">(1a34bb7)</span></li>
<li>generators/poppler/generator_pdf.cpp <span style="color: grey">(a363842)</span></li>
<li>generators/spectre/generator_ghostview.h <span style="color: grey">(ab9c33d)</span></li>
<li>generators/spectre/generator_ghostview.cpp <span style="color: grey">(7b0146a)</span></li>
<li>generators/tiff/generator_tiff.h <span style="color: grey">(9ad1de6)</span></li>
<li>generators/tiff/generator_tiff.cpp <span style="color: grey">(bc692de)</span></li>
<li>generators/xps/generator_xps.h <span style="color: grey">(4c56e8c)</span></li>
<li>generators/xps/generator_xps.cpp <span style="color: grey">(2399a96)</span></li>
<li>part.cpp <span style="color: grey">(67440bd)</span></li>
<li>ui/presentationwidget.cpp <span style="color: grey">(4aaec2a)</span></li>
<li>ui/propertiesdialog.cpp <span style="color: grey">(485c2de)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/118293/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>