<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [PATCH] Feature : extract media in html format"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56843#c11">Comment # 11</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [PATCH] Feature : extract media in html format"
href="https://bugs.freedesktop.org/show_bug.cgi?id=56843">bug 56843</a>
from <span class="vcard"><a class="email" href="mailto:tsdgeos@terra.es" title="Albert Astals Cid <tsdgeos@terra.es>"> <span class="fn">Albert Astals Cid</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=56843#c8">comment #8</a>)
<span class="quote">> > Also can you explain the code change in poppler/FileSpec.cc?
>
> I made the change in the file "FileSpec.cc" because of all the test files
> I used the old method to read the filename does not work.
> Looking at the struture of PDF, I noticed that the tag 'F' was still present.</span >
Why it wouldn't work, it first checks for UF and if it's not there uses F,
besides the specification clearly documents that F is deprecated and that UF
should be used if both are present
Also i'm going to guess this is one of your first open source contributions and
you are making the mistake of dumping everything in the same patch, i.e.
[PATCH 6/7] Remove "
does not have anything to do with "Feature : extract media in html format" and
you should submit it separately because dumping everything in the same patch
makes it harder to review.
- if (ignore||(complexMode && !xml)) {
+ if (ignore|| !(complexMode && withMedia)) {
Why the !xml was removed?
delete assets; is not enough, check how other goolists are deleted
I'm also a bit confused with
+ type = richMediaContentType;
configurations = new GooList();
assets = new GooList();
//Type name (Optional; ExtensionLevel 3)
if (dict->lookup("Type", &obj1)->isName()) {
- type = obj1.getName();
+ const char *name = obj1.getName();
+ if(!strcmp("RichMediaContent", name)){
+ type = richMediaContentType;
+ }
}
if the type will always be richMediaContentType, why bother asking the dict?
Also it would be cool if you could squash patch 7 into patch 3, this would make
reviewing easier</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>