unrtf - Patches: patch #7676, unrtf as a library (huge)
You are not allowed to post comments on this tracker with your current authentication level.
patch #7676: unrtf as a library (huge)
Submitter: | Wolfgang Sourdeau <wolfgang> | ||
Submitted: | Thu 24 Nov 2011 02:49:56 PM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | None | Privacy: | Public |
Assigned to: | None | Open/Closed: | Open |
Fri 07 Sep 2012 04:50:42 PM UTC, comment #1: |
Alexey Dobrov <adobrov> |
Thu 24 Nov 2011 02:49:56 PM UTC, original submission:
For the SOGo/OpenChange project, we are now using your converter for interpreting RTF content. In order to do so, I have modified the code of unrtf in order to let it built as a library. Mainly, this implied converting all global variables into members of structures that are passed from function to function. The structures in questions have been layed out as this: one for the conversion parameters inherited from the command line, another for all internal processing as well as the pre-existing one for the output personality.
|
Wolfgang Sourdeau <wolfgang> |
Depends on the following items: None found
Items that depend on this one: None found
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.
Follows 1 latest change.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2011-11-24 | wolfgang | Attached File | - | ![]() |
Added unrtf-0.21.2.diff, #24441 |
I have used your patch and proceeded with making a library from unrtf, see https://savannah.gnu.org/patch/index.php?7846 .
There were some mistakes: when you moved global variables to function arguments, you have forgotten to pass them in several places. That was leading to unpredictable behavior and sometimes even to segmentation faults. I tried to correct these mistakes, but I'm not sure that I have found all of them. However, thanks a lot for your patch -- it was very useful for my purposes!