From 31ced30c1ea413e4b0b2569fb97e480280e921c6 Mon Sep 17 00:00:00 2001 From: Peter Johnson Date: Sun, 22 Jul 2018 15:07:22 -0700 Subject: [PATCH] HttpParser: Change Execute() to return StringRef (#1209) This is more user-friendly than returning the number of parsed bytes. --- wpiutil/src/main/native/include/wpi/HttpParser.h | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/wpiutil/src/main/native/include/wpi/HttpParser.h b/wpiutil/src/main/native/include/wpi/HttpParser.h index 8a09fe82a8..3ff91c94c4 100644 --- a/wpiutil/src/main/native/include/wpi/HttpParser.h +++ b/wpiutil/src/main/native/include/wpi/HttpParser.h @@ -48,10 +48,11 @@ class HttpParser { /** * Executes the parser. An empty input is treated as EOF. * @param in input data - * @return Number of parsed bytes. + * @return Trailing input data after the parse. */ - size_t Execute(StringRef in) { - return http_parser_execute(&m_parser, &m_settings, in.data(), in.size()); + StringRef Execute(StringRef in) { + return in.drop_front( + http_parser_execute(&m_parser, &m_settings, in.data(), in.size())); } /**