On 8/25/05, David Crossley <crossley@apache.org> wrote:
> Diwaker Gupta wrote:
> > When I say XML, I mean all kinds of XML files (XSL, *.fv, *.ft)
>
> How will the tool know which are xml file-types?
> We have a multitude of xml filename extensions.
> There is a list on one of the tools in the
> "committers" svn repository at relicense/src/insert.pl
I don't see why this is a problem. I'll just write a simple script
that runs the tool on whatever extensions we use for XML files. Tidy
doesn't know/care about the file extension -- you just feed it some
input, and it gives you the output.
> I think that it is doing too much, e.g. removing the
> blank lines before major elements, e.g. <xsl:template>
Like I said, its configurable. We can pick and choose. Since this is
an automated process, I'm not too worried about the tool doing "too
much" -- just change the config, re run and voila!
> We should start with very simple stuff, e.g. just tabs
> and trailing whitespace, then gradually add other operations.
> However we don't want to get too strict on code style.
Agreed.
> I suggest that we define a list of what we would possibly
> want to adrress. Here is a start:
>
> 1 whitespace at end-of-line
> 2 tabs to four-space
You mean 2 spaces.
> 3 indentation
Ok.
> 4 word-wrap for long lines
What would be a good length? Right now I use 80.
> 5 whitespace between attribute definitions
Ok.
> We need to at least do 1 and 2 while 5 may be doubtful.
>
> It seems to be too vigorous with wrapping. Perhaps wider
> would be better.
> < <xsl:key name="h5s" match="h5" use="generate-id(preceding-sibling::h4[1])"/>
> ---
> > <xsl:key name="h5s" match="h5"
> > use="generate-id(preceding-sibling::h4[1])" />
It won't wrap if there are no spaces to break the line at. It won't
wrap between attribute values (I think this is configurable though). I
usually use 80 in my editor as well, so this works for me. We can set
it to 100 or something else if that works for more people.
> Also we need to run it on one of the xdocs
> in site-author/content/xdocs to see what it does
> with word-wrapping for long lines of element content.
I'll do this soon.
--
Web/Blog/Gallery: floatingsun.net
|