Fix bug in document.correctEncoding() which failed to properly parse xml doctypes.

README

@@ -20,10 +20,11 @@
 
  Getting the package up and running is simple enough:
  
- $ cd /path/to/code/dir
+    $ goinstall github.com/jteeuwen/go-pkg-xmlx
- $ git clone http://github.com/jteeuwen/go-pkg-xmlx.git
+
- $ cd go-pkg-xmlx
+ Using it:
- $ make
+
+    import xmlx "github.com/jteeuwen/go-pkg-xmlx"
 
 ================================================================================
  API

document.go

@@ -240,24 +240,34 @@ loop:
 			if err == os.EOF {
 				break loop
 			}
+			
 			return "", err
 		}
 
 		switch tt := tok.(type) {
 		case xml.ProcInst:
 			if tt.Target == "xml" { // xml doctype
-				enc = strings.ToLower(string(tt.Inst))
+				var pair []string
-				if i := strings.Index(enc, `encoding="`); i > -1 {
+				var entry string
-					enc = enc[i+len(`encoding="`):]
+
-					i = strings.Index(enc, `"`)
+				list := strings.Split(string(tt.Inst), " ", -1)
-					enc = enc[:i]
+				for _, entry = range list {
-					break loop
+					if pair = strings.Split(entry, "=", -1); len(pair) < 2 {
+						continue
+					}
+
+					switch pair[0] {
+					case "encoding":
+						enc = pair[1][1:len(pair[1])-1]
+						break loop
+					}
 				}
 			}
 		}
 	}
 
 	if enc == "utf-8" {
+		// Data already in utf-8 format. Nothing to do here.
 		return data, nil
 	}
 

xmlx_test.go

@@ -27,7 +27,7 @@ func TestLoadLocal(t *testing.T) {
 func TestLoadRemote(t *testing.T) {
 	doc := New()
 
-	if err := doc.LoadUri("http://rss.cnn.com/rss/cnn_latest.rss"); err != nil {
+	if err := doc.LoadUri("http://blog.golang.org/feeds/posts/default"); err != nil {
 		t.Error(err.String())
 		return
 	}