bugunrtf - Bugs: bug #34805, Tables not converting properly in...

 
 

bug #34805: Tables not converting properly in unrtf 0.21.2

Submitter:  None
Submitted:  Sat 12 Nov 2011 12:04:26 AM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Fixed Privacy:  Public
Assigned to:  None Originator Email:  -email is unavailable-
Open/Closed:  Closed Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Tue 16 Dec 2014 04:21:21 PM UTC, comment #1: 



I am attaching the patch against current code corresponding to this change.


(file #32682)

Jean-Francois Dockes <medoc>
Group administrator
Sat 12 Nov 2011 12:04:26 AM UTC, original submission:  

Hey all,

There was a bug in version 0.21.2 (and, I think, previous versions as well).  Basically, the \intbl command wasn't initiating the creation of a table until a new group (i.e. a segment in RTF starting with {blah blah ...}) proceeded it.  This should not be the case - if the \intbl command is processed, it should trigger the start of a table (if not already within a table) without requiring a new group to follow the command.

I have fixed this bug, and attached the file I modified with the bug fix (convert.c).  I've tested this fix with a bunch of RTF files, and it seems to work well.  However, I haven't done any extensive testing (and I'm not sure what kind of requirements there are for testing a bug fix for GNU software).

Anyway, I don't know where to officially submit this fix, so I'm putting it here.  Hopefully that is okay.

Cheers!

Jarrett

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #24356:  convert.c added by None (112KiB - text/x-csrc - Patched convert.c file)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by daved (Updated the item)
  • -email is unavailable- added by medoc (Updated the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2014-12-28 daved StatusNone Fixed
        Open/ClosedOpen Closed
    2014-12-16 medoc Attached File- Added convert-check_for_table.patch, #32682
    2011-11-12 None Attached File- Added convert.c, #24356

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code