NME output not reprouce DD token

Bug #1091758 reported by r
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Nyctergatis Markup Engine
Fix Released
Low
Yves Piguet

Bug Description

The attached demo converts Creole to Creole and should produce the same output as input. However, with DD tokens, it outputs a semicolon (;) instead of a colon (:).

Input for NMEOutputFormatNME conversion:

; one
: two

Wrong output of NMEOutputFormatNME conversion:

; one
; two

Expected output of NMEOutputFormatNME conversion:

; one
: two

Revision history for this message
r (ralfjunker) wrote :
Revision history for this message
Yves Piguet (yves-piguet) wrote :

The bug can also be reproduced with the conversion tool:
nme --nme
;a
:b
<ctrl>-D

Changed in nme:
importance: Undecided → Low
status: New → Confirmed
assignee: nobody → Yves Piguet (yves-piguet)
Changed in nme:
status: Confirmed → Fix Committed
Revision history for this message
r (ralfjunker) wrote : Re: [Bug 1091758] Re: NME output not reprouce DD token

Fix confirmed for DT and DD on separate lines:

; one
: two

However, with DT and DD on the SAME line:

; one : two

a second fix is required. Block starts at NME.c, line 3318:

    case kNMETokenDD:
        context.level = context.nesting;
        CheckError(flushStyleTags(styleStack, &styleNesting,
                i0,
                outputFormat, &context));
        CheckError(addEndPar(FALSE, outputFormat, &context, i0));
        context.listNum[context.nesting - 1] = kNMEListNumDD; // ADD
        HOOK(parHookFun, context.level, context.item, TRUE, ";:");
        if (!NMEAddString(outputFormat->beginDD, -1,
                context.ctrlChar, &context))
            return kNMEErrNotEnoughMemory;
        CheckError(checkWordwrap(&context, outputFormat));
        context.level = 0;
        // DEL context.listNum[context.nesting - 1] = kNMEListNumDD;
        break;

Ralf

On 21.12.2012 16:43, Yves Piguet wrote:

> ** Changed in: nme
> Status: Confirmed => Fix Committed

Changed in nme:
status: Fix Committed → In Progress
Revision history for this message
Yves Piguet (yves-piguet) wrote :

Fix committed, thanks!

Changed in nme:
status: In Progress → Fix Committed
Changed in nme:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.