You are viewing a plain text version of this content. The canonical link for it is here.
Posted to doxia-dev@maven.apache.org by Vincent Siveton <vi...@gmail.com> on 2007/07/12 13:48:08 UTC

Re: svn commit: r555559 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java

Hi Lukas,

We need also to update the APT format docu for \f in Doxia site.

Moreover, we have 2 docs which describe the apt format: one in maven
and one in doxia. WDYT to remove the doc from the maven site and to
add an Apache rule to go to the doxia one?

Cheers,

Vincent

2007/7/12, ltheussl@apache.org <lt...@apache.org>:
> Author: ltheussl
> Date: Thu Jul 12 03:03:51 2007
> New Revision: 555559
>
> URL: http://svn.apache.org/viewvc?view=rev&rev=555559
> Log:
> Use \f for pagebreak
>
> Modified:
>     maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
>
> Modified: maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java
> URL: http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java?view=diff&rev=555559&r1=555558&r2=555559
> ==============================================================================
> --- maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java (original)
> +++ maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java Thu Jul 12 03:03:51 2007
> @@ -332,8 +332,7 @@
>
>      public void pageBreak()
>      {
> -       // TODO: APT parse defect... pagebreak is never used.
> -       write( EOL + "^L" + EOL );
> +        write( EOL + "\f" + EOL );
>      }
>
>      public void paragraph()
>
>
>

Re: svn commit: r555559 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java

Posted by Dennis Lundberg <de...@apache.org>.
Vincent Siveton wrote:
> Hi Lukas,
> 
> We need also to update the APT format docu for \f in Doxia site.
> 
> Moreover, we have 2 docs which describe the apt format: one in maven
> and one in doxia. WDYT to remove the doc from the maven site and to
> add an Apache rule to go to the doxia one?

+1 to redirect it to doxia

> Cheers,
> 
> Vincent
> 
> 2007/7/12, ltheussl@apache.org <lt...@apache.org>:
>> Author: ltheussl
>> Date: Thu Jul 12 03:03:51 2007
>> New Revision: 555559
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=555559
>> Log:
>> Use \f for pagebreak
>>
>> Modified:
>>     
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>>
>>
>> Modified: 
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>>
>> URL: 
>> http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java?view=diff&rev=555559&r1=555558&r2=555559 
>>
>> ============================================================================== 
>>
>> --- 
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>> (original)
>> +++ 
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>> Thu Jul 12 03:03:51 2007
>> @@ -332,8 +332,7 @@
>>
>>      public void pageBreak()
>>      {
>> -       // TODO: APT parse defect... pagebreak is never used.
>> -       write( EOL + "^L" + EOL );
>> +        write( EOL + "\f" + EOL );
>>      }
>>
>>      public void paragraph()
>>
>>
>>


-- 
Dennis Lundberg

Re: svn commit: r555559 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java

Posted by Vincent Siveton <vi...@gmail.com>.
2007/7/12, Lukas Theussl <lt...@apache.org>:
> well, ^L (Control-L) is the symbol for a form feed character, which is
> the only sequence that aptconvert recognizes as a page break (not "^L"
> as a string, nor "\f"), so I think the docs are correct.

Ok

> I'd prefer to leave the page at
> http://maven.apache.org/guides/mini/guide-apt-format.html, just remove
> everything after "Document structure" and add a link instead to the
> doxia site, after the short introductory paragraph that explains what
> apt is.
>

Agree but a lot of license header in APT files use the following link
http://maven.apache.org/guides/mini/guide-apt-format.html
So I think a redirection is better.

Cheers,

Vincent


[SNIP]

Re: svn commit: r555559 - /maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java

Posted by Lukas Theussl <lt...@apache.org>.
well, ^L (Control-L) is the symbol for a form feed character, which is 
the only sequence that aptconvert recognizes as a page break (not "^L" 
as a string, nor "\f"), so I think the docs are correct.

I'd prefer to leave the page at 
http://maven.apache.org/guides/mini/guide-apt-format.html, just remove 
everything after "Document structure" and add a link instead to the 
doxia site, after the short introductory paragraph that explains what 
apt is.

WDYT?

-Lukas


Vincent Siveton wrote:
> Hi Lukas,
> 
> We need also to update the APT format docu for \f in Doxia site.
> 
> Moreover, we have 2 docs which describe the apt format: one in maven
> and one in doxia. WDYT to remove the doc from the maven site and to
> add an Apache rule to go to the doxia one?
> 
> Cheers,
> 
> Vincent
> 
> 2007/7/12, ltheussl@apache.org <lt...@apache.org>:
> 
>> Author: ltheussl
>> Date: Thu Jul 12 03:03:51 2007
>> New Revision: 555559
>>
>> URL: http://svn.apache.org/viewvc?view=rev&rev=555559
>> Log:
>> Use \f for pagebreak
>>
>> Modified:
>>     
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>>
>>
>> Modified: 
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>>
>> URL: 
>> http://svn.apache.org/viewvc/maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java?view=diff&rev=555559&r1=555558&r2=555559 
>>
>> ============================================================================== 
>>
>> --- 
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>> (original)
>> +++ 
>> maven/doxia/doxia/trunk/doxia-modules/doxia-module-apt/src/main/java/org/apache/maven/doxia/module/apt/AptSink.java 
>> Thu Jul 12 03:03:51 2007
>> @@ -332,8 +332,7 @@
>>
>>      public void pageBreak()
>>      {
>> -       // TODO: APT parse defect... pagebreak is never used.
>> -       write( EOL + "^L" + EOL );
>> +        write( EOL + "\f" + EOL );
>>      }
>>
>>      public void paragraph()
>>
>>
>>