gergely/taskwarrior-2.x
1
0
Fork 0
Code Issues Actions Packages Projects Releases 1 Wiki Activity

Nibbler: Remove unused function argument

Browse Source
This commit is contained in:
Wilhelm Schuermann
2015-11-12 13:36:23 +01:00
parent 0961a011f4
commit 96ca7eb5f2
4 changed files with 11 additions and 29 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/JSON.cpp
View File

@@ -66,7 +66,7 @@ json::string::string (const std::string& other)
json::string* json::string::parse (Nibbler& nibbler)
{
std::string value;
if (nibbler.getQuoted ('"', value, false))
if (nibbler.getQuoted ('"', value))
{
json::string* s = new json::string ();
s->_data = value;
@@ -303,7 +303,7 @@ bool json::object::parse_pair (
{
Nibbler n (nibbler);
if (n.getQuoted ('"', name, false))
if (n.getQuoted ('"', name))
{
n.skipWS ();
if (n.skip (':'))

8
src/Nibbler.cpp
View File

@@ -180,10 +180,7 @@ bool Nibbler::getN (const int quantity, std::string& result)
}
////////////////////////////////////////////////////////////////////////////////
bool Nibbler::getQuoted (
char c,
std::string& result,
bool quote /* = false */)
bool Nibbler::getQuoted (char c, std::string& result)
{
bool inquote = false;
bool inescape = false;
@@ -210,9 +207,6 @@ bool Nibbler::getQuoted (
if (current == c && !inescape)
{
if (quote)
result += current;
if (!inquote)
{
inquote = true;

2
src/Nibbler.h
View File

@@ -48,7 +48,7 @@ public:
bool getUntilEOS (std::string&);
bool getN (const int, std::string&);
bool getQuoted (char, std::string&, bool quote = false);
bool getQuoted (char, std::string&);
bool getDigit (int&);
bool getDigit4 (int&);
bool getDigit3 (int&);

26
test/nibbler.t.cpp
View File

@@ -35,7 +35,7 @@ Context context;
////////////////////////////////////////////////////////////////////////////////
int main (int, char**)
{
UnitTest t (240);
UnitTest t (236);
// Ensure environment has no influence.
unsetenv ("TASKDATA");
@@ -167,28 +167,16 @@ int main (int, char**)
t.notok (n.getQuoted ('\'', s), "\"one\\\"two\" : getQuoted (''') -> false");
n = Nibbler ("\"one\\\"two\"");
t.ok (n.getQuoted ('"', s, false), "\"one\\\"two\" : getQuoted ('\"', false, false) -> true");
t.is (s, "one\\\"two", "getQuoted ('\"', false) -> one\\\"two");
t.ok (n.getQuoted ('"', s), "\"one\\\"two\" : getQuoted ('\"', s) -> true");
t.is (s, "one\\\"two", "getQuoted ('\"', s) -> one\\\"two");
n = Nibbler ("\"one\\\"two\"");
t.ok (n.getQuoted ('"', s, true), "\"one\\\"two\" : getQuoted ('\"', false, true) -> true");
t.is (s, "\"one\\\"two\"", "getQuoted ('\"', true) -> \"one\\\"two\"");
n = Nibbler ("\"one\\\"two\"");
t.ok (n.getQuoted ('"', s, false), "\"one\\\"two\" : getQuoted ('\"', true, false) -> true");
t.is (s, "one\\\"two", "getQuoted ('\"', false) -> one\\\"two");
n = Nibbler ("\"one\\\"two\"");
t.ok (n.getQuoted ('"', s, true), "\"one\\\"two\" : getQuoted ('\"', s, true) -> true");
t.is (s, "\"one\\\"two\"", "getQuoted ('\"', s, true) -> \"one\\\"two\"");
t.ok (n.getQuoted ('"', s), "\"one\\\"two\" : getQuoted ('\"', s) -> true");
t.is (s, "one\\\"two", "getQuoted ('\"', s) -> one\\\"two");
n = Nibbler ("\"one\\\\\"");
t.ok (n.getQuoted ('\"', s, true), "\"one\\\" : getQuoted ('\"', s, true) -> true");
t.is (s, "\"one\\\\\"", "getQuoted ('\"', s, true) -> \"one\\\\\"");
n = Nibbler ("\"one\\\\\"");
t.ok (n.getQuoted ('\"', s, false), "one\\ : getQuoted ('\"', s, false) -> true");
t.is (s, "one\\\\", "getQuoted ('\"', s, false) -> \"one\\\\\"");
t.ok (n.getQuoted ('\"', s), "one\\ : getQuoted ('\"', s) -> true");
t.is (s, "one\\\\", "getQuoted ('\"', s) -> \"one\\\\\"");
// bool getDigit (int&);
t.diag ("Nibbler::getDigit");