[liblouis-liblouisxml] Re: [liblouisutdml] 5 new revisions pushed by mwhapples on 2014-06-04 13:48 GMT
- From: Brandon Roller <brandon.r.roller@xxxxxxxxx>
- To: liblouis-liblouisxml@xxxxxxxxxxxxx
- Date: Thu, 5 Jun 2014 08:24:46 -0400
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> 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> wrote:
>
>> When you send the fragment do you include the DTD reference?
>>
>>
>>
>>
>>
>> *From:* 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
>> *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> 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
>>
>>
>>
>
>
>
Other related posts:
