Fixes #3: The reserved xml prefix may be used without pre-declaring…

… it.

README.md

@@ -140,10 +140,15 @@ In this case no table will have a `parent` attribute, elements will not have the
 - No support for extended (Unicode) characters in element/attribute names
 - No support for charset
 - No support for [XInclude](http://www.w3.org/TR/xinclude/)
+- Does not ensure that the reserved `xml` prefix is never redefined to an illegal namespace
+- Does not ensure that the reserved `xmlns` prefix is never used as an element prefix
 
 
 ## History
 
+### v0.5.3 2014-Feb-12
++ Fixes Issue #3: The [reserved `xml` prefix](http://www.w3.org/TR/xml-names/#ns-decl) may be used without pre-declaring it. (Thanks David Durkee.)
+
 ### v0.5.2 2013-Nov-7
 + Lua 5.2 compatible
 + Parser now errors if it finishes without finding a root element,

slaxml.lua

@@ -1,5 +1,5 @@
 --[=====================================================================[
-v0.5.2 Copyright © 2013 Gavin Kistner <!@phrogz.net>; MIT Licensed
+v0.5.3 Copyright © 2013 Gavin Kistner <!@phrogz.net>; MIT Licensed
 See http://github.com/Phrogz/SLAXML for details.
 --]=====================================================================]
 local SLAXML = {
@@ -85,6 +85,7 @@ function SLAXML:parse(xml,options)
 	end
 
 	local function nsForPrefix(prefix)
+		if prefix=='xml' then return 'http://www.w3.org/XML/1998/namespace' end -- http://www.w3.org/TR/xml-names/#ns-decl
 		for i=#nsStack,1,-1 do if nsStack[i][prefix] then return nsStack[i][prefix] end end
 		error(("Cannot find namespace for prefix %s"):format(prefix))
 	end

test/test.lua

@@ -4,7 +4,7 @@ local SLAXML = require 'slaxdom'
 require 'io'
 require 'lunity'
 
-module( 'TEST_LXSC', lunity )
+module( 'TEST_SLAXML', lunity )
 
 local XML = {}
 for filename in io.popen('ls files'):lines() do
@@ -125,6 +125,16 @@ function test_dom_entities()
 	assertEqual(t.kids[6].value,' your code > all ')
 end
 
+function test_xml_namespace()
+	local doc = SLAXML:dom(XML['xml_namespace'])
+	for i,attr in ipairs(doc.root.attr) do
+		if attr.name=='space' then
+			assertEqual(attr.nsURI,[[http://www.w3.org/XML/1998/namespace]])
+			break
+		end
+	end
+end
+
 function test_dom_namespaces()
 	local scxmlNS  = "http://www.w3.org/2005/07/scxml"
 	local phrogzNS = "http://phrogz.net/"