diff --git a/dom.d b/dom.d index 47f939c..952f45c 100644 --- a/dom.d +++ b/dom.d @@ -2109,8 +2109,7 @@ class Document { pos++; string v = htmlEntitiesDecode(data[start..pos], strict); pos++; // skip over the end - return v; - break; + return v; default: if(strict) parseError("Attributes must be quoted"); @@ -2227,8 +2226,7 @@ class Document { if(!caseSensitive) tname = tname.toLower; - return Ele(1, null, tname); // closing tag reports itself here - break; + return Ele(1, null, tname); // closing tag reports itself here case ' ': // assume it isn't a real element... if(strict) parseError("bad markup - improperly placed <"); @@ -2351,10 +2349,8 @@ class Document { default: assert(0); case '/': // self closing tag return addTag(true); - break; case '>': return addTag(false); - break; case ' ': case '\t': case '\n': @@ -2365,11 +2361,8 @@ class Document { switch(data[pos]) { case '/': // self closing tag return addTag(true); - break; case '>': // closed tag; open -- we now read the contents - - return addTag(false); - break; + return addTag(false); default: // it is an attribute string attrName = readAttributeName(); string attrValue = attrName; @@ -2637,7 +2630,7 @@ class Document { string prolog = "\n"; ///. - string toString() const { + override string toString() const { return prolog ~ root.toString(); } @@ -3166,13 +3159,7 @@ int intFromHex(string hex) { current.evenChild = true; break; - case "visited": - case "active": - case "hover": - case "target": - case "focus": - case "checked": - case "selected": + case "visited", "active", "hover", "target", "focus", "checked", "selected": current.attributesPresent ~= "nothing"; // FIXME /* @@ -3183,15 +3170,15 @@ int intFromHex(string hex) { // extensions not implemented //case "text": // takes the text in the element and wraps it in an element, returning it +/ - - case "before": - case "after": + goto case; + case "before", "after": current.attributesPresent ~= "FIXME"; break; default: //if(token.indexOf("lang") == -1) //assert(0, token); + break; } state = State.Starting; break; @@ -3213,7 +3200,7 @@ int intFromHex(string hex) { } break; } - // fallthrough + goto case; case State.ExpectingAttributeCloser: if(token != "]") { // not the closer; consider it part of comparison @@ -3502,7 +3489,7 @@ class CssStyle { } ///. - string toString() { + override string toString() { string ret; if(originatingRule.length) ret = originatingRule ~ " {"; diff --git a/web.d b/web.d index 250bcb2..b8608e1 100644 --- a/web.d +++ b/web.d @@ -168,7 +168,6 @@ class ApiProvider { Document _defaultPage() { throw new Exception("no default"); - return null; } Element _getGenericContainer() @@ -1660,10 +1659,12 @@ void formatAs(T, R)(T ret, R api, ref JSONValue returnValue, string format, stri case "table": auto document = new Document(""); static if(__traits(compiles, structToTable(document, ret))) + { returnValue.str = structToTable(document, ret).toString(); + break; + } else goto badType; - break; default: badType: throw new Exception("Couldn't get result as " ~ format);