[liblouis-liblouisxml] Re: [liblouisutdml] 5 new revisions pushed by mwhapples on 2014-06-04 13:48 GMT
- From: "John J. Boyer" <john.boyer@xxxxxxxxxxxxxxxxx>
- To: liblouis-liblouisxml@xxxxxxxxxxxxx
- Date: Thu, 5 Jun 2014 08:40:44 -0500
When using translateString with utd you might want to include
"paragraphs no" in the configuration settings string.
John
On Thu, Jun 05, 2014 at 01:32:32PM +0100, Michael Whapples wrote:
> One thing I notice in what you posted is that the brl element when
> using translateString has a newpage and newline element at the
> beginning, where as the file sample does not. Might the fact its at
> the beginning of a page be impacting on this?
>
> Would be useful to have the full XML being passed in to both functions.
>
> Michael Whapples
> On 05/06/2014 13:24, Brandon Roller wrote:
> >Yes, but when I look at the liblouisutdml logs I see the following
> >line: "Begin insert_text: node->content= Items one."
> >The space is included in the log statement.
> >
> >
> >On Thu, Jun 5, 2014 at 4:51 AM, Michael Whapples
> ><dmarc-noreply@xxxxxxxxxxxxx <mailto:dmarc-noreply@xxxxxxxxxxxxx>>
> >wrote:
> >
> > My initial reaction was the same as Keith, DTD information must be
> > included for it to work correct.
> >
> > The only other one where I would expect things to be different is
> > if you give anything but nimas in, as I only turned on
> > internetAccess yes in nimas.cfg.
> >
> > However if the XML is identical in both calls then there should be
> > no difference. Thus it seems like a bug. I will get to look at
> > this, could you open a ticket in RT, feel free to assign it to me
> > as I probably will be picking it up anyway.
> >
> > Michael Whapples
> > On 04/06/2014 19:00, Brandon Roller wrote:
> >> I just tried that and it didn't work.
> >>
> >>
> >> On Wed, Jun 4, 2014 at 1:49 PM, Keith Creasy <kcreasy@xxxxxxx
> >> <mailto:kcreasy@xxxxxxx>> wrote:
> >>
> >> When you send the fragment do you include the DTD reference?
> >>
> >> *From:*liblouis-liblouisxml-bounce@xxxxxxxxxxxxx
> >> <mailto:liblouis-liblouisxml-bounce@xxxxxxxxxxxxx>
> >> [mailto:liblouis-liblouisxml-bounce@xxxxxxxxxxxxx
> >> <mailto:liblouis-liblouisxml-bounce@xxxxxxxxxxxxx>] *On
> >> Behalf Of *Brandon Roller
> >> *Sent:* Wednesday, June 04, 2014 1:37 PM
> >> *To:* liblouis-liblouisxml@xxxxxxxxxxxxx
> >> <mailto:liblouis-liblouisxml@xxxxxxxxxxxxx>
> >> *Subject:* [liblouis-liblouisxml] Re: [liblouisutdml] 5 new
> >> revisions pushed by mwhapples on 2014-06-04 13:48 GMT
> >>
> >> The changes to whitespace seem to be working well, which is
> >> great, but it doesn't seem to be honored when translateString
> >> is called. Any ideas why this might be happening? For
> >> example, if the original element is translated as follows,
> >> which is correct, the braille translation is correct: <span
> >> semantics="action,no"> Item one.<brl index="0 1 1 2 3 4 5 6 6
> >> 9"> ,item "o4</brl></span>
> >>
> >> However, if I call translateString and pass that element the
> >> result does not include the space as follows: <span
> >> semantics="action,no"> Items one.<brl index="0 0 1 2 3 4 5 6
> >> 6 9"><newpage brlnumber="1" /><newline xy="20,10" />,items
> >> "o4</brl></span>.
> >>
> >> On Wed, Jun 4, 2014 at 9:48 AM, <liblouisutdml@xxxxxxxxxxxxxx
> >> <mailto:liblouisutdml@xxxxxxxxxxxxxx>> wrote:
> >>
> >> 5 new revisions:
> >>
> >> Revision: acac8f8318d5
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Mon Jun 2 11:53:48 2014 UTC
> >> Log: Add logging to show nodes as they are parsed.
> >> http://code.google.com/p/liblouisutdml/source/detail?r=acac8f8318d5
> >>
> >> Revision: 0fe61e0119e2
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Mon Jun 2 12:47:54 2014 UTC
> >> Log: Explicit cast.
> >> http://code.google.com/p/liblouisutdml/source/detail?r=0fe61e0119e2
> >>
> >> Revision: 09f7c4b43c60
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Mon Jun 2 12:52:44 2014 UTC
> >> Log: Move log message to after all variable declarations.
> >> http://code.google.com/p/liblouisutdml/source/detail?r=09f7c4b43c60
> >>
> >> Revision: 5a1c5e095e7b
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Tue Jun 3 08:22:15 2014 UTC
> >> Log: More logging
> >> http://code.google.com/p/liblouisutdml/source/detail?r=5a1c5e095e7b
> >>
> >> Revision: 71378ffa2b31
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Wed Jun 4 13:53:29 2014 UTC
> >> Log: Disable whitespace stripping
> >> http://code.google.com/p/liblouisutdml/source/detail?r=71378ffa2b31
> >>
> >>
> >> ==============================================================================
> >> Revision: acac8f8318d5
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Mon Jun 2 11:53:48 2014 UTC
> >> Log: Add logging to show nodes as they are parsed.
> >> http://code.google.com/p/liblouisutdml/source/detail?r=acac8f8318d5
> >>
> >> Modified:
> >> /liblouisutdml/examine_document.c
> >>
> >> =======================================
> >> --- /liblouisutdml/examine_document.c Sat Mar 30 05:19:40
> >> 2013 UTC
> >> +++ /liblouisutdml/examine_document.c Mon Jun 2 11:53:48
> >> 2014 UTC
> >> @@ -41,6 +41,7 @@
> >> examine_document (xmlNode * node)
> >> {
> >> /*Examine the parse tree, add semantic attributes and set
> >> indicators.*/
> >> + logMessage(LOG_INFO, "Begin examine_document:
> >> node->name=%s", node->name);
> >> xmlNode *child;
> >> if (node == NULL)
> >> return 0;
> >> @@ -111,6 +112,7 @@
> >> examText (xmlNode * node)
> >> /*We may want to examine text content in the future*/
> >> {
> >> + logMessage(LOG_INFO, "Begin examText: node->content=%s",
> >> node->content);
> >> switch (ud->stack[ud->top])
> >> {
> >> case pagenum:
> >>
> >>
> >> ==============================================================================
> >> Revision: 0fe61e0119e2
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Mon Jun 2 12:47:54 2014 UTC
> >> Log: Explicit cast.
> >> http://code.google.com/p/liblouisutdml/source/detail?r=0fe61e0119e2
> >>
> >> Modified:
> >> /liblouisutdml/examine_document.c
> >>
> >> =======================================
> >> --- /liblouisutdml/examine_document.c Mon Jun 2 11:53:48
> >> 2014 UTC
> >> +++ /liblouisutdml/examine_document.c Mon Jun 2 12:47:54
> >> 2014 UTC
> >> @@ -41,7 +41,7 @@
> >> examine_document (xmlNode * node)
> >> {
> >> /*Examine the parse tree, add semantic attributes and set
> >> indicators.*/
> >> - logMessage(LOG_INFO, "Begin examine_document:
> >> node->name=%s", node->name);
> >> + logMessage(LOG_INFO, "Begin examine_document:
> >> node->name=%s", (char *)node->name);
> >> xmlNode *child;
> >> if (node == NULL)
> >> return 0;
> >>
> >>
> >> ==============================================================================
> >> Revision: 09f7c4b43c60
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Mon Jun 2 12:52:44 2014 UTC
> >> Log: Move log message to after all variable declarations.
> >> http://code.google.com/p/liblouisutdml/source/detail?r=09f7c4b43c60
> >>
> >> Modified:
> >> /liblouisutdml/examine_document.c
> >>
> >> =======================================
> >> --- /liblouisutdml/examine_document.c Mon Jun 2 12:47:54
> >> 2014 UTC
> >> +++ /liblouisutdml/examine_document.c Mon Jun 2 12:52:44
> >> 2014 UTC
> >> @@ -41,8 +41,8 @@
> >> examine_document (xmlNode * node)
> >> {
> >> /*Examine the parse tree, add semantic attributes and set
> >> indicators.*/
> >> - logMessage(LOG_INFO, "Begin examine_document:
> >> node->name=%s", (char *)node->name);
> >> xmlNode *child;
> >> + logMessage(LOG_INFO, "Begin examine_document:
> >> node->name=%s", node->name);
> >> if (node == NULL)
> >> return 0;
> >> ud->stack[++ud->top] = set_sem_attr (node);
> >>
> >>
> >> ==============================================================================
> >> Revision: 5a1c5e095e7b
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Tue Jun 3 08:22:15 2014 UTC
> >> Log: More logging
> >> http://code.google.com/p/liblouisutdml/source/detail?r=5a1c5e095e7b
> >>
> >> Modified:
> >> /liblouisutdml/examine_document.c
> >>
> >> =======================================
> >> --- /liblouisutdml/examine_document.c Mon Jun 2 12:52:44
> >> 2014 UTC
> >> +++ /liblouisutdml/examine_document.c Tue Jun 3 08:22:15
> >> 2014 UTC
> >> @@ -125,6 +125,7 @@
> >> static void
> >> examCdataa (xmlNode * node)
> >> {
> >> + logMessage(LOG_INFO, "Begin examCdata");
> >> ud->has_cdata = 1;
> >> }
> >>
> >>
> >>
> >> ==============================================================================
> >> Revision: 71378ffa2b31
> >> Branch: default
> >> Author: Michael Whapples
> >> Date: Wed Jun 4 13:53:29 2014 UTC
> >> Log: Disable whitespace stripping
> >> http://code.google.com/p/liblouisutdml/source/detail?r=71378ffa2b31
> >>
> >> Modified:
> >> /liblouisutdml/transcriber.c
> >>
> >> =======================================
> >> --- /liblouisutdml/transcriber.c Mon Jun 2 08:45:16 2014 UTC
> >> +++ /liblouisutdml/transcriber.c Wed Jun 4 13:53:29 2014 UTC
> >> @@ -1095,16 +1095,16 @@
> >> int k;
> >> // int stripSpace = 0;
> >> logMessage(LOG_INFO, "Begin insert_text:
> >> node->content=%s", node->content);
> >> - for (k = length; k > 0 && node->content[k - 1] <= 32; k--);
> >> + // for (k = length; k > 0 && node->content[k - 1] <= 32; k--);
> >> // We want to track if the node only contains space 0x20
> >> characters
> >> // if (node->content[k - 1] != 32)
> >> // stripSpace = 1;
> >> // if (stripSpace == 0)
> >> // k = length; // We want to keep the spaces
> >> - if (k <= 0)
> >> - return;
> >> - if (k < length)
> >> - length = k + 1; /*Keep last whitespace */
> >> + //if (k <= 0)
> >> + // return;
> >> + //if (k < length)
> >> + // length = k + 1; /*Keep last whitespace */
> >> if (style->emphasis)
> >> push_action (style->emphasis);
> >> if (ud->format_for == utd)
> >> For a description of the software, to download it and links to
> >> project pages go to http://www.abilitiessoft.com
> >>
> >>
> >
> >
>
--
John J. Boyer; President, Chief Software Developer
Abilitiessoft, Inc.
http://www.abilitiessoft.com
Madison, Wisconsin USA
Developing software for people with disabilities
For a description of the software, to download it and links to
project pages go to http://www.abilitiessoft.com
Other related posts:
