summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRobert Griesemer <gri@golang.org>2009-12-28 17:24:53 -0800
committerRobert Griesemer <gri@golang.org>2009-12-28 17:24:53 -0800
commit66b13bd9dcf2105bb58b8246ccdfc0a16310b588 (patch)
tree0d27bc5dba1fe402a7cea1d3f7c032d91973a36a
parent61bcf9d2bd8726e6734a7292e12967e91017f597 (diff)
downloadgo-66b13bd9dcf2105bb58b8246ccdfc0a16310b588.tar.gz
Fix bug in godoc tab conversion filter:
tabs after an empty line where not converted. Also, made it more robust in the presence of (unexpected) ' ' and '\v' chars in indentation mode. R=r CC=golang-dev http://codereview.appspot.com/181085
-rw-r--r--src/cmd/godoc/godoc.go18
1 files changed, 13 insertions, 5 deletions
diff --git a/src/cmd/godoc/godoc.go b/src/cmd/godoc/godoc.go
index a381c3a58..8b6bc5e46 100644
--- a/src/cmd/godoc/godoc.go
+++ b/src/cmd/godoc/godoc.go
@@ -471,8 +471,8 @@ type tconv struct {
}
-func (p *tconv) writeIndent(n int) (err os.Error) {
- i := n * *tabwidth
+func (p *tconv) writeIndent() (err os.Error) {
+ i := p.indent
for i > len(spaces) {
i -= len(spaces)
if _, err = p.output.Write(spaces); err != nil {
@@ -490,12 +490,20 @@ func (p *tconv) Write(data []byte) (n int, err os.Error) {
for n, b = range data {
switch p.state {
case indenting:
- if b == '\t' {
+ switch b {
+ case '\t', '\v':
+ p.indent += *tabwidth
+ case '\n':
+ p.indent = 0
+ if _, err = p.output.Write(data[n : n+1]); err != nil {
+ return
+ }
+ case ' ':
p.indent++
- } else {
+ default:
p.state = collecting
pos = n
- if err = p.writeIndent(p.indent); err != nil {
+ if err = p.writeIndent(); err != nil {
return
}
}