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

2010-04-19 Thread gerel
> Date: Mon, 19 Apr 2010 14:51:42 +0200 (CEST) > From: Jose E. Marchesi > > > Ok. Here is try 2 :-) > > + if (cpos<0){ > +return __pdf_fsys_disk_get_status_from_errno (errno); > + }else{ > +*pos = cpos; > +return PDF_OK;

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

2010-04-19 Thread Jose E . Marchesi
> Same problem with the codying style. Are you sure you sent the > right patch? Ah, sorry. It was late night :-) Applied. Now all tests pass successfully in my machine. Many thanks :) -- Jose E. Marchesijema...@gnu.org GNU Project http://www.gnu.org

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

2010-04-19 Thread gerel
> Date: Mon, 19 Apr 2010 14:51:42 +0200 (CEST) > From: Jose E. Marchesi > > > Ok. Here is try 2 :-) > > + if (cpos<0){ > +return __pdf_fsys_disk_get_status_from_errno (errno); > + }else{ > +*pos = cpos; > +return PDF_OK;

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

2010-04-19 Thread Vincent Torri
On Mon, 19 Apr 2010, Jose E. Marchesi wrote: Ok. Here is try 2 :-) + if (cpos<0){ +return __pdf_fsys_disk_get_status_from_errno (errno); + }else{ +*pos = cpos; +return PDF_OK; + } Same problem with the codying style. Are you sure

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

2010-04-19 Thread Jose E . Marchesi
Ok. Here is try 2 :-) + if (cpos<0){ +return __pdf_fsys_disk_get_status_from_errno (errno); + }else{ +*pos = cpos; +return PDF_OK; + } Same problem with the codying style. Are you sure you sent the right patch? -- Jose E. M

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

2010-04-18 Thread gerel
Ok. Here is try 2 :-) ## # Bazaar merge directive format 2 (Bazaar 0.90) # revision_id: ge...@gnu.org-20100418215209-0imic49ywr2s6dx9 # target_branch: file:///home/gerel/PROJECTS/libgnupdf/trunk/ # testament_sha1: e23fd060d388ce397f3749dd265260573d26eafb # timestamp: 2010-04-18 18:52:31 -0300 # b

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

2010-04-18 Thread Jose E . Marchesi
Hi Gerel. + long cpos; + cpos = ftell(((pdf_fsys_disk_file_t)file->data)->file_descriptor); Please put a blank character between the function names and the parenthesized parameters list. -- Jose E. Marchesi http://www.jemarch.net GNU Project http://ww

[pdf-devel] Patch for FS#114

2010-04-17 Thread gerel
Hi pdf-fanatics, I wrote a patch for FS#114. The test 'pdf_stm_bseek_003' which was failing, now passes with this patch. ## # Bazaar merge directive format 2 (Bazaar 0.90) # revision_id: ge...@gnu.org-20100417202736-pwl7xtabdsz66804 # target_branch: file:///home/gerel/PROJECTS/libgnupdf/trunk/ #