forks
/
arsd
mirror of https://github.com/adamdruppe/arsd.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed warnings in dom.d and web.d

This commit is contained in:
Abscissa 2011-07-21 18:33:10 -04:00
parent 2907e290d2
commit da4610db32
2 changed files with 13 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
dom.d
View File

@ -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 = "<!DOCTYPE html>\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 ~ " {";

5
web.d
View File

@ -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("<root></root>");
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);