debian-packages/dyaml
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Moved a branch outside of aloop in scanPlain() to improve performance.

Browse source
This commit is contained in:
Ferdinand Majerech 2014-08-05 20:58:05 +02:00
parent 57d936ed0f
commit d505728824
1 changed files with 20 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
source/dyaml/scanner.d
View file

@ -1739,17 +1739,29 @@ final class Scanner
while(reader_.peekByte() != '#') while(reader_.peekByte() != '#')
{ {
// Scan the entire plain scalar. // Scan the entire plain scalar.
uint length = 0; size_t length = 0;
dchar c; dchar c = void;
// Moved the if() out of the loop for optimization.
if(flowLevel_ == 0)
{
c = reader_.peek(length);
for(;;)
{
const cNext = reader_.peek(length + 1);
if(search.canFind(c) || (c == ':' && search.canFind(cNext))) { break; }
++length;
c = cNext;
}
}
else
{
for(;;) for(;;)
{ {
c = reader_.peek(length); c = reader_.peek(length);
const bool done = search.canFind(c) || (flowLevel_ == 0 && c == ':' && if(search.canFind(c) || ",:?[]{}"d.canFind(c)) { break; }
search.canFind(reader_.peek(length + 1))) ||
(flowLevel_ > 0 && ",:?[]{}"d.canFind(c));
if(done) { break; }
++length; ++length;
} }
}
// It's not clear what we should do with ':' in the flow context. // It's not clear what we should do with ':' in the flow context.
if(flowLevel_ > 0 && c == ':' && if(flowLevel_ > 0 && c == ':' &&