Bug 105151

Why did you remove the setLinkDestination(movie); call?

(In reply to Albert Astals Cid from comment #1)
> Why did you remove the setLinkDestination(movie); call?

It was surrounded by #if 0 /* ... */ #endif and hence seemed to be dead code.

-    QCOMPARE( page->search(QString("non-ascii:"), rectLeft, rectTop, rectRight, rectBottom, Poppler::Page::FromTop, Poppler::Page::CaseSensitive), true );
+    QCOMPARE( page->search(QString("non-ascii:"), rectLeft, rectTop, rectRight, rectBottom, Poppler::Page::FromTop, Poppler::Page::IgnoreCase), true );

is wrong?

Created attachment 137454 [details] [review]
Second version of pathc to reduce compiler warnings

(In reply to Albert Astals Cid from comment #3)
> -    QCOMPARE( page->search(QString("non-ascii:"), rectLeft, rectTop,
> rectRight, rectBottom, Poppler::Page::FromTop,
> Poppler::Page::CaseSensitive), true );
> +    QCOMPARE( page->search(QString("non-ascii:"), rectLeft, rectTop,
> rectRight, rectBottom, Poppler::Page::FromTop, Poppler::Page::IgnoreCase),
> true );
> 
> is wrong?

Indeed, thanks for spotting it. Fixed that in an updated version of the patch. (It seems that the test is actually not dependent on that flag but still a stronger check in the original version.)

Can you rebase the patch?

tsdgeos@xps:~/devel/poppler:master$ LANG=C git am attachment.cgi\?id\=137454
Applying: Fix buffer size warning due to missing space for null terminator in pdfseparate.
Applying: Fix warnings due to the use of deprecated overloads of Poppler::Page::Search in tests of Qt5 frontend.
error: patch failed: qt5/tests/check_search.cpp:21
error: qt5/tests/check_search.cpp: patch does not apply
Patch failed at 0002 Fix warnings due to the use of deprecated overloads of Poppler::Page::Search in tests of Qt5 frontend.
The copy of the patch that failed is found in: .git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".


Seems like the encoding of the file is not utf8 and the special characters are getting confused?

(In reply to Albert Astals Cid from comment #6)
> Can you rebase the patch?

[...]

> Seems like the encoding of the file is not utf8 and the special characters
> are getting confused?

Yes, one of the edited files was not UTF-8 encoded. The rebase does not change anything, but I think I could have messed up the patch file as I needed to copy-and-paste it from Kate due to a Tumbleweed update killing the kfile kio plug-in and with it all KDE file dialogs. Sorry about that.

I will reattach the newly generated patch file directly.

Created attachment 137491 [details] [review]
Patch to reduce compiler warnings

to be honest i'm not totally sure i agree with with the last one, but anyway, i've pushed it.