Re: [pdf-devel] Patch for FS#116

2010-04-19 Thread Jose E . Marchesi
> Ok. Here is a patch with Aleks' fixes and formatting standards. > Jose, this one would be OK to apply Applied. -- Jose E. Marchesijema...@gnu.org GNU Project http://www.gnu.org

Re: [pdf-devel] Patch for FS#116

2010-04-19 Thread Aleksander Morgado
> > > >/* 0x0100 */ > > -if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ > > - (pdf_text_filter_remove_line_endings(text) != PDF_OK)) > > + if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ > > + (pdf_text_filter_remove_line_endings(text

Re: [pdf-devel] Patch for FS#116

2010-04-18 Thread gerel
> Date: Sun, 18 Apr 2010 13:02:32 +0200 (CEST) > From: Jose E. Marchesi > > >/* 0x0100 */ > -if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ > - (pdf_text_filter_remove_line_endings(text) != PDF_OK)) > + if((filter & PDF_TEXT_FILTER_REMOVE_LINE_

Re: [pdf-devel] Patch for FS#116

2010-04-18 Thread gerel
Here is try 2. Review and apply. ## # Bazaar merge directive format 2 (Bazaar 0.90) # revision_id: ge...@gnu.org-20100418214759-c1ho9719elvho1kk # target_branch: file:///home/gerel/PROJECTS/libgnupdf/trunk/ # testament_sha1: ef2e62b091722280c7473af3bcc5769214eadcfb # timestamp: 2010-04-18 18:48:0

Re: [pdf-devel] Patch for FS#116

2010-04-18 Thread Aleksander Morgado
> /* 0x0100 */ >-if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ >- (pdf_text_filter_remove_line_endings(text) != PDF_OK)) >+ if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ >+ (pdf_text_filter_remove_line_endings(text) != PDF_OK)) > > Please d

Re: [pdf-devel] Patch for FS#116

2010-04-18 Thread Jose E . Marchesi
/* 0x0100 */ -if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ - (pdf_text_filter_remove_line_endings(text) != PDF_OK)) + if((filter & PDF_TEXT_FILTER_REMOVE_LINE_ENDINGS) && \ + (pdf_text_filter_remove_line_endings(text) != PDF_OK)) Please don't wr

Re: [pdf-devel] Patch for FS#116

2010-04-18 Thread Aleksander Morgado
Hey gerel, > Here is a patch for FS#116. New pdf_text_get_printable. > Aleks, tell me if this was what you meant. Just one comment, below. Thanks! > > +const pdf_char_t * > +pdf_text_get_printable (pdf_text_t text) > +{ > + pdf_size_t size; > + > + if (text->printable != NULL){ > +if (te

[pdf-devel] Patch for FS#116

2010-04-17 Thread gerel
Hi GNUsuperblovers. Here is a patch for FS#116. New pdf_text_get_printable. Aleks, tell me if this was what you meant. ## # Bazaar merge directive format 2 (Bazaar 0.90) # revision_id: ge...@gnu.org-20100417224914-sh8jq4wz0vee17tr # target_branch: file:///home/gerel/PROJECTS/libgnupdf/trunk/ # t