-
-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RTF document model #5
Comments
Hi again, I have begun implementing. My thoughts were that it is best to design a builder pattern. The core builder implementation would only work with interfaces, and there would be a default implementation which constructs a document model. As such, it would be possible to connect a completely different document model directly to the builder, parsing an RTF straight into an unknown document model. This would be achieved by writing thin wrappers implementing the interfaces defined by RTF Parser Kit in terms of the foreign document model. At the same time, the default RTF Parser Kit document model implementation could be used to transfer a foreign model and then write an RTF. But maybe there is an even better way, I have not yet started on that part. For now, I have text landing in the right destinations and will start looking into styling next. Regards, |
Sounds good! Sorry I'm slow at replying... and thanks for the pull request, I'll get that merged as soon as I can. I really appreciate you taking the time to help out! Jon |
At the moment, I have imported the RTF Parser Kit code into my large application and am developing the changes there. That's mainly because I wanted to see first how well it all worked out, before separating the code and changing the build system to deal with one more dependency (also, I still target Java 5). But when you have some time to review my code, just tell me and I'll transfer the changes into the repository which I forked on GitHub. Regards, |
Hi, just to let you know I am making progress... rewrote my additions today and it's beginning to take shape. Just curious: Will you have any time to implement the writing RTFs part? Best regards, |
You can see my current work here: https://github.com/stippi/rtfparserkit/tree/document-builder ... though I realized after publishing that I forgot to fix the license headers to also use the APL. Sorry about that, will do at the next syncing point. |
Hi Jon, |
Hey Jon, really no feedback at all in all this time? :-( |
Hi Jon,
thanks for making this nice piece of code. I am trying to use it in my project, but the StandardRtfParser is still too low level for my purposes. I am pondering whether to contribute to your project and start on an RTF document model, especially, since it would allow to create an RTF writer as well. I have some ideas how to approach this, but I would love to discuss with you first and find out what you had in mind. Do you think the issue tracker is the ideal place for discussion? If so we could use this issue, otherwise we could switch to other means of communication. For example, I think chatting in IRC could be beneficial, then noting the result back here. My nick on Freenode is stippi.
The text was updated successfully, but these errors were encountered: