aboutsummaryrefslogtreecommitdiff
path: root/bpkg/manifest-parser.cxx
diff options
context:
space:
mode:
Diffstat (limited to 'bpkg/manifest-parser.cxx')
-rw-r--r--bpkg/manifest-parser.cxx111
1 files changed, 20 insertions, 91 deletions
diff --git a/bpkg/manifest-parser.cxx b/bpkg/manifest-parser.cxx
index f8a2e36..513573a 100644
--- a/bpkg/manifest-parser.cxx
+++ b/bpkg/manifest-parser.cxx
@@ -18,7 +18,7 @@ namespace bpkg
next ()
{
if (s_ == end)
- return name_value {"", "", l_, c_, l_, c_};
+ return name_value {"", "", line, column, line, column};
xchar c (skip_spaces ());
@@ -36,8 +36,7 @@ namespace bpkg
if (s_ == body && c == ':')
{
s_ = start;
- uint64_t ln (c.line ()), cn (c.column ());
- return name_value {"", "", ln, cn, ln, cn};
+ return name_value {"", "", c.line, c.column, c.line, c.column};
}
// Regardless of the state, what should come next is a name,
@@ -49,13 +48,12 @@ namespace bpkg
skip_spaces ();
c = get ();
- if (is_eos (c))
+ if (eos (c))
{
// This is ok as long as the name is empty.
//
if (!r.name.empty ())
- throw parsing (name_, c.line (), c.column (),
- "':' expected after name");
+ throw parsing (name_, c.line, c.column, "':' expected after name");
s_ = end;
@@ -67,7 +65,7 @@ namespace bpkg
}
if (c != ':')
- throw parsing (name_, c.line (), c.column (), "':' expected after name");
+ throw parsing (name_, c.line, c.column, "':' expected after name");
skip_spaces ();
parse_value (r);
@@ -76,7 +74,7 @@ namespace bpkg
// The character after the value should be either a newline or eos.
//
- assert (c == '\n' || is_eos (c));
+ assert (c == '\n' || eos (c));
if (c == '\n')
get ();
@@ -132,10 +130,10 @@ namespace bpkg
{
xchar c (peek ());
- r.name_line = c.line ();
- r.name_column = c.column ();
+ r.name_line = c.line;
+ r.name_column = c.column;
- for (; !is_eos (c); c = peek ())
+ for (; !eos (c); c = peek ())
{
if (c == ':' || c == ' ' || c == '\t' || c == '\n')
break;
@@ -150,8 +148,8 @@ namespace bpkg
{
xchar c (peek ());
- r.value_line = c.line ();
- r.value_column = c.column ();
+ r.value_line = c.line;
+ r.value_column = c.column;
string& v (r.value);
string::size_type n (0); // Size of last non-space character (simple mode).
@@ -170,7 +168,7 @@ namespace bpkg
c = peek ();
ml = true;
}
- else if (is_eos (p))
+ else if (eos (p))
ml = true;
else
unget (c);
@@ -180,7 +178,7 @@ namespace bpkg
// newline", that is, a newline that was part of the milti-line mode
// introductor or an escape sequence.
//
- for (bool nl (ml); !is_eos (c); c = peek ())
+ for (bool nl (ml); !eos (c); c = peek ())
{
// Detect the special "\n\\\n" sequence. In the multi-line mode,
// this is a "terminator". In the simple mode, this is a way to
@@ -209,7 +207,7 @@ namespace bpkg
get ();
xchar c1 (peek ());
- if (c1 == '\n' || is_eos (c1))
+ if (c1 == '\n' || eos (c1))
{
if (ml)
break;
@@ -240,7 +238,7 @@ namespace bpkg
get ();
xchar c2 (peek ());
- if (c2 == '\n' || is_eos (c2))
+ if (c2 == '\n' || eos (c2))
break;
else
{
@@ -264,7 +262,7 @@ namespace bpkg
get ();
xchar c1 (peek ());
- if (c1 == '\n' || is_eos (c1))
+ if (c1 == '\n' || eos (c1))
{
if (c1 == '\n')
{
@@ -278,7 +276,7 @@ namespace bpkg
get ();
xchar c2 (peek ());
- if (c2 == '\n' || is_eos (c1))
+ if (c2 == '\n' || eos (c1))
{
v += '\\';
n = v.size ();
@@ -314,9 +312,9 @@ namespace bpkg
skip_spaces ()
{
xchar c (peek ());
- bool start (c.column () == 1);
+ bool start (c.column == 1);
- for (; !is_eos (c); c = peek ())
+ for (; !eos (c); c = peek ())
{
switch (c)
{
@@ -344,7 +342,7 @@ namespace bpkg
// Read until newline or eos.
//
- for (c = peek (); !is_eos (c) && c != '\n'; c = peek ())
+ for (c = peek (); !eos (c) && c != '\n'; c = peek ())
get ();
continue;
@@ -359,75 +357,6 @@ namespace bpkg
return c;
}
- // Character interface.
- //
-
- manifest_parser::xchar manifest_parser::
- peek ()
- {
- if (unget_)
- return buf_;
- else
- {
- if (eos_)
- return xchar (xchar::traits_type::eof (), l_, c_);
- else
- {
- xchar::int_type v (is_.peek ());
-
- if (v == xchar::traits_type::eof ())
- eos_ = true;
-
- return xchar (v, l_, c_);
- }
- }
- }
-
- manifest_parser::xchar manifest_parser::
- get ()
- {
- if (unget_)
- {
- unget_ = false;
- return buf_;
- }
- else
- {
- // When is_.get () returns eof, the failbit is also set (stupid,
- // isn't?) which may trigger an exception. To work around this
- // we will call peek() first and only call get() if it is not
- // eof. But we can only call peek() on eof once; any subsequent
- // calls will spoil the failbit (even more stupid).
- //
- xchar c (peek ());
-
- if (!is_eos (c))
- {
- is_.get ();
-
- if (c == '\n')
- {
- l_++;
- c_ = 1;
- }
- else
- c_++;
- }
-
- return c;
- }
- }
-
- void manifest_parser::
- unget (const xchar& c)
- {
- // Because iostream::unget cannot work once eos is reached,
- // we have to provide our own implementation.
- //
- buf_ = c;
- unget_ = true;
- }
-
// manifest_parsing
//