[Bug 740506]

2016-03-05 Thread Carlos Garcia Campos
\o/ Great job guys!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to poppler in Ubuntu.
https://bugs.launchpad.net/bugs/740506

Title:
  verify digital signatures

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/740506/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1085526]

2016-03-05 Thread Carlos Garcia Campos
\o/ Great job guys!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1085526

Title:
  ubuntu pdf doc viewer will not let me sign a document

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/1085526/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 33288]

2016-02-20 Thread Carlos Garcia Campos
Comment on attachment 121848
Cache result of inner loop in visitDepthFirst

Looks good to me, pushed. Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/33288

Title:
  Evince doesn't handle columns properly

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/33288/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2016-01-16 Thread Carlos Garcia Campos
As far as I remember the fix only worked for some versions of GS.
Anyway, I'll try to find some time too look at this in detail again.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 116453]

2015-01-27 Thread Carlos Garcia Campos
(In reply to Jason Crain from comment #6)
 I suppose if I add an option to findText, I should also add a flag
 (POPPLER_FIND_IGNORE_COMBINING?) to PopplerFindFlags, for the glib front
 end's poppler_page_find_text_with_options().  It would be nice if someone
 could confirm that evince would actually use this option.

I don't see a reason why someone might want to search for ü and not find
a word containing ü. So, if there are two methods in poppler core, I
would change the glib bindings to use the one correctly finding
combining characters.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to poppler in Ubuntu.
https://bugs.launchpad.net/bugs/116453

Title:
  evince can not find ü in attached PDF

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/116453/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs

[Bug 1242678]

2014-11-20 Thread Carlos Garcia Campos
(In reply to Pacho Ramos from comment #36)
 (In reply to Carlos Garcia Campos from comment #35)
  I've reverted the patch in master for now, until we find a proper solution.
 
 Do you know when a new tarball is planned including this and other fixes?
 Thanks

No plans, but I can make a new release once this problem is fixed, for
example.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-09-04 Thread Carlos Garcia Campos
Still doesn't work here, it seems to rotate the first page only of a
multipage document.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-09-04 Thread Carlos Garcia Campos
I've reverted the patch in master for now, until we find a proper
solution.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-08-14 Thread Carlos Garcia Campos
(In reply to comment #27)
 (In reply to comment #25)
  Created attachment 101589 [details] [review] [review]
  Change direction of rotation
  
  The promised patch.
 
 I'm not sure this should be done here or in evince. Libspectre should do
 what the PS spec says, if evince wants the opposite, evince should do the
 conversion. Or am I misunderstanding the patch?

in any case we should document the expected behaviour in the
documentation of spectre_render_context_get/set_rotation

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-08-14 Thread Carlos Garcia Campos
(In reply to comment #25)
 Created attachment 101589 [details] [review]
 Change direction of rotation
 
 The promised patch.

I'm not sure this should be done here or in evince. Libspectre should do
what the PS spec says, if evince wants the opposite, evince should do
the conversion. Or am I misunderstanding the patch?

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-08-14 Thread Carlos Garcia Campos
we can discuss them here, if this patch has caused regressions we might
revert it if we don't find a new solution for the regressions.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-06-25 Thread Carlos Garcia Campos
(In reply to comment #22)
 (In reply to comment #20)
  (In reply to comment #19)
   Thanks Marek! Do you still plan to look at the evince side as well (since
   the rotation seems to still be wrong when using it, as mentioned in 
   previous
   comments)
  
  Hi Sebastien,
  
  thank you for reminding me this. I'll look at that but not now since I have
  several tasks I have to finish before I can jump on this.
  Maybe at the end of the next week.
 
 I've filled a bug for this here:
 https://bugzilla.gnome.org/show_bug.cgi?id=731786. A patch is attached there
 which solves it for me.
 
 
 (In reply to comment #21)
  I've been trying this and I'm getting the opposite results, the attached ps
  file is rotated without the patch but nothing is rendered when rotated with
  the patch.
 
 I've just tried it with evince from master with patched libspectre and it
 works for me as expected :(.

Maybe it has to do with the gs version?

$ gs --version
9.05

From Debian testing.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-06-18 Thread Carlos Garcia Campos
I've been trying this and I'm getting the opposite results, the attached
ps file is rotated without the patch but nothing is rendered when
rotated with the patch.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-06-11 Thread Carlos Garcia Campos
(In reply to comment #17)
 Created attachment 100723 [details] [review]
 Rotate documents correctly
 
 Hi,
 
 I'm sorry for my late response, I was super busy with something else.

No problem!

 (In reply to comment #15)
  Comment on attachment 99331 [details] [review] [review]
  Rotate documents correctly
  
  Review of attachment 99331 [details] [review] [review]:
  -
  
  This looks good to me, except the confusing NONE/LANDSCAPE thing. Adrian,
  does the PostScript part looks good to you?
  
  ::: libspectre/spectre-device.c
  @@ +206,4 @@
 return SPECTRE_STATUS_RENDER_ERROR;
 }

   + if (rc-orientation == NONE || rc-orientation == LANDSCAPE) {
  
  Why NONE and LANDSCAPE? shouldn't we invert width/height when orientation is
  LANDSCAPE and SEASCAPE? Ah, I know what's going on, you are using the
  internal values of the parser (my fault, I think I suggested it) but
  rc-orientation is actually a SpectreOrientation enum value, so you should
  use SPECTRE_ORIENTATION_PORTRAIT || SPECTRE_ORIENTATION_REVERSE_PORTRAIT
  that are 0 and 2 like NONE and LANDSCAPE in the internal parser.
  
  @@ +269,4 @@
 args[arg++] = -dNOPLATFONTS;

 if (rc-width != -1  rc-height != -1) {
   + if (rc-orientation == NONE || rc-orientation == LANDSCAPE) {
  
  Same here.
 
 My brain was probably on a vacation :). I've fixed this so that it uses the
 SPECTRE_ORIENTATION_PORTRAIT and SPECTRE_ORIENTATION_REVERSE_PORTRAIT.

:-)

 Regards
 
 Marek

Pushed to git master, thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-06-04 Thread Carlos Garcia Campos
Comment on attachment 99331
Rotate documents correctly

Review of attachment 99331:
-

This looks good to me, except the confusing NONE/LANDSCAPE thing.
Adrian, does the PostScript part looks good to you?

::: libspectre/spectre-device.c
@@ +206,4 @@
   return SPECTRE_STATUS_RENDER_ERROR;
   }
  
 + if (rc-orientation == NONE || rc-orientation == LANDSCAPE) {

Why NONE and LANDSCAPE? shouldn't we invert width/height when
orientation is LANDSCAPE and SEASCAPE? Ah, I know what's going on, you
are using the internal values of the parser (my fault, I think I
suggested it) but rc-orientation is actually a SpectreOrientation enum
value, so you should use SPECTRE_ORIENTATION_PORTRAIT ||
SPECTRE_ORIENTATION_REVERSE_PORTRAIT that are 0 and 2 like NONE and
LANDSCAPE in the internal parser.

@@ +269,4 @@
   args[arg++] = -dNOPLATFONTS;
  
   if (rc-width != -1  rc-height != -1) {
 + if (rc-orientation == NONE || rc-orientation == LANDSCAPE) {

Same here.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-05-21 Thread Carlos Garcia Campos
I'm not a PostScript expert either.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-05-21 Thread Carlos Garcia Campos
(In reply to comment #12)
 Yes, you can replace setpagedevice with pop. It is not clear to me how your
 patch is doing this. Something like /setpagedevice { pop } bind def should
 work.

Thanks Adrian. It seems Marek submitted the same patch again instead of
the updated one.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 1242678]

2014-05-15 Thread Carlos Garcia Campos
Comment on attachment 96165
Rotate documents correctly

Review of attachment 96165:
-

Thanks for the patch!

::: libspectre/spectre-device.c
@@ +206,5 @@
   return SPECTRE_STATUS_RENDER_ERROR;
   }
  
 + scaled_width = (int) ((width * rc-x_scale) + 0.5);
 + scaled_height = (int) ((height * rc-y_scale) + 0.5);

I think it would be clearer if we set these values depending on the
orientation, and then we just use scale_width, scaled_height below.

@@ +226,5 @@
  rc-text_alpha_bits);
   args[arg++] = graph_alpha = _spectre_strdup_printf 
 (-dGraphicsAlphaBits=%d,
   
 rc-graphic_alpha_bits);
 +
 + if (rc-orientation == 0 || rc-orientation == 2) {

Don't use magic numbers here, use NONE and LANDSCAPE instead.

@@ +232,5 @@
 + args[arg++] = resolution = _spectre_strdup_printf (-r%fx%f,
 +rc-x_scale 
 * rc-x_dpi,
 +rc-y_scale 
 * rc-y_dpi);
 + }
 + else {

} else {

::: libspectre/spectre-gs.c
@@ +237,5 @@
  
 + switch (rotation) {
 + default:
 + tmp_xoffset = xoffset + x;
 + tmp_yoffset = yoffset + y;

I don't think we need the tmp_ variables, we can just modify the
existing xoffset/yoffset.

@@ +277,5 @@
doc-endsetup))
   return FALSE;
  
 + if (rotation != 0) {
 + set = _spectre_strdup_printf (%d rotate, rotation);

Where does this end up? After the setup and before the pages? Isn't it
recommended to apply the rotation after the translation? I think we
could move this to spectre_gs_process, after the translate. We pass the
rotation to spectre_gs_process and when it's != NONE we inject the
rotate command there. What do you think?

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/1242678

Title:
  evince cannot render some EPS files

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/1242678/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 33288]

2013-11-26 Thread Carlos Garcia Campos
While working on bug #71160 I've found another regression introduced by
this fix. In some cases, additional lines are added to the selection.
For example, open the hig document and go to the first page. Start
selecting the second line, but dragging from the margin, and you will
see that the first line is selected too. This is because the second line
is more indented than the first one. This fix changed the way blocks and
lines are included in the selection by using the manhattan distance, and
in this case, the distance of the first line is less than the second
line, but the first line doesn't even intersect with the selection
rectangle. If you start the selection closer to the beginning of the
second line, then the first line is not included because distance to the
second line is less in such case.

You can play with it now using the text demo of poppler-glib-demo. I've
added an area selector to get the text of a given area. Try using X1=0,
Y1=122, which should discard the first line, but it doesn't. However
using X1=257, Y1=122 discards the first line entirely.

So, I think that we need to somehow discard blocks and lines that don't
intersect with the selection rectangle even if the manhattan distance is
less than any other block/line.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/33288

Title:
  Evince doesn't handle columns properly

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/33288/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 667752]

2012-11-26 Thread Carlos Garcia Campos
The problem with the patch is that in some cases it's clear that it
improves the output, but in some others I have doubts whether the output
is better or worse, I'll attach some examples found while running my
tests.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/667752

Title:
  Non-embedded standard fonts in PDF files are not displayed/rendered
  correctly

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/667752/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 667752]

2012-11-26 Thread Carlos Garcia Campos
Created attachment 70366
Test results examples

Let me know if you want me to send the PDFs to you

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to evince in Ubuntu.
https://bugs.launchpad.net/bugs/667752

Title:
  Non-embedded standard fonts in PDF files are not displayed/rendered
  correctly

To manage notifications about this bug go to:
https://bugs.launchpad.net/evince/+bug/667752/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 740506]

2012-09-17 Thread Carlos Garcia Campos
(In reply to comment #26)
 What's missing in gnutls is a way to parse all the relevant components of the
 PKCS#7 object as present in a PDF signature.
 
 It seems that in gnutls they assume those objects can only contain 
 certificates
 and CRLs as you can confirm if you go through the functions that take
 gnutls_pkcs7_t as argument.
 
 With openssl you can get the certificates, signature, and the digest of the
 signed content (these are the essential parts for detached signatures as used
 in PDF) as well as any optional timestamps or CRLs.

Would it be a lot of work to add support for that to gnutls?

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to poppler in Ubuntu.
https://bugs.launchpad.net/bugs/740506

Title:
  verify digital signatures

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/740506/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 740506]

2012-09-17 Thread Carlos Garcia Campos
(In reply to comment #30)
 (In reply to comment #26)
  What's missing in gnutls is a way to parse all the relevant components of 
  the
  PKCS#7 object as present in a PDF signature.
  
  It seems that in gnutls they assume those objects can only contain 
  certificates
  and CRLs as you can confirm if you go through the functions that take
  gnutls_pkcs7_t as argument.
  
  With openssl you can get the certificates, signature, and the digest of the
  signed content (these are the essential parts for detached signatures as 
  used
  in PDF) as well as any optional timestamps or CRLs.
 
 Would it be a lot of work to add support for that to gnutls?

Replying to myself:

KaL I wonder if it could be useful for glib-networking to implement the 
missing things in gnutls, or if we don't need that at all
danw reading...
danw chpe, KaL_out: both gnutls and glib-networking intentionally only do 
TLS, not crypto in general, so I don't think it makes sense to add the extra 
PKCS#7 functionality to either of them
danw NSS would be better than OpenSSL, and once all the p11-kit / 
NSS-shared-DB stuff gets figured out fully, then NSS-based apps will be able to 
access your gnome-keyring certificates via PKCS#11

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to poppler in Ubuntu.
https://bugs.launchpad.net/bugs/740506

Title:
  verify digital signatures

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/740506/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 39890]

2012-09-17 Thread Carlos Garcia Campos
(In reply to comment #49)
 Oh, wait, didn't realize the functions it's adding new parameters are private,
 ignore my breaks API part.

No problem :-) but please, ask me before committing any change that
might break the public glib API even in master.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/39890

Title:
  characters change when selecting text

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/39890/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 39890]

2012-09-17 Thread Carlos Garcia Campos
(In reply to comment #47)
 I've commited this patch to master (what will be 0.22) and not to 0.20 since 
 it
 breaks the public glib api

Really? why? I agree with not committing it to the stable branch, but I
don't think the patch breaks the API, or am I missing something?
Breaking the API should be avoided even in master.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/39890

Title:
  characters change when selecting text

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/39890/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 33288]

2012-09-10 Thread Carlos Garcia Campos
(In reply to comment #42)
 Created an attachment (id=31406) [details]
 3/5 reading order (bug fixed)

It seems this bug introduced an important performance regression, see
the detailed analysis in poppler mailing list:

http://lists.freedesktop.org/archives/poppler/2010-October/006566.html

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/33288

Title:
  Evince doesn't handle columns properly

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/33288/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 39890]

2012-08-22 Thread Carlos Garcia Campos
(In reply to comment #29)

 Note i have *not* verified it fixes the problem mentioned in this bug. Carlos
 can you do that part?

it's weird, I don't see any difference with and without the patch with
poppler-glib-demo and the minimal example attached to this bug.

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/39890

Title:
  characters change when selecting text

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/39890/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 39890]

2012-08-15 Thread Carlos Garcia Campos
(In reply to comment #25)
 Created attachment 65449 [details] [review]
 Allow multiple fonts in a TextWord
 
 This patch modifies TextWord so that each character can be a different font. 
 Function poppler_page_get_text_attributes is updated to allow for attributes 
 to
 change mid-word.

Thanks for the patch!, Albert could you run the regtests for text
backend with this patch, please?

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/39890

Title:
  characters change when selecting text

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/39890/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 39890]

2012-08-15 Thread Carlos Garcia Campos
Sure, no hurry, thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to the bug report.
https://bugs.launchpad.net/bugs/39890

Title:
  characters change when selecting text

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/39890/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 711300]

2012-01-23 Thread Carlos Garcia Campos
Comment on attachment 55629
fix ctm when painting groups

Review of attachment 55629:
-

Patch looks good to me and it doesn't introduce regressions in my tests,
so please, push it to both branches. Thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is subscribed to poppler in Ubuntu.
https://bugs.launchpad.net/bugs/711300

Title:
  rotated included PDF

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/711300/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs


[Bug 193927]

2012-01-21 Thread Carlos Garcia Campos
Comment on attachment 55785
Fix-test-for-rotation.patch

Review of attachment 55785:
-

Pushed to git master, thanks!

-- 
You received this bug notification because you are a member of Ubuntu
Desktop Bugs, which is a bug assignee.
https://bugs.launchpad.net/bugs/193927

Title:
  Evince not displaying pdf correctly

To manage notifications about this bug go to:
https://bugs.launchpad.net/poppler/+bug/193927/+subscriptions

-- 
desktop-bugs mailing list
desktop-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/desktop-bugs