Introduced lower footprint HttpVersion instead of String based.

Author: QuickSander

README.md

@@ -1,6 +1,11 @@
 # ArduinoHttpServer
 
 [![Build Status](https://travis-ci.org/QuickSander/ArduinoHttpServer.svg?branch=master)](https://travis-ci.org/QuickSander/ArduinoHttpServer)
+[![Code Climate](https://codeclimate.com/github/QuickSander/ArduinoHttpServer/badges/gpa.svg)](https://codeclimate.com/github/QuickSander/ArduinoHttpServer)
+[![Test Coverage](https://codeclimate.com/github/QuickSander/ArduinoHttpServer/badges/coverage.svg)](https://codeclimate.com/github/QuickSander/ArduinoHttpServer/coverage)
+
+
+
 
 *Server side minimalistic Object Oriented HTTP protocol implementation for the Arduino platform.*
 
@@ -34,6 +39,11 @@ ArduinoHttpServer::StreamHttpReply httpReply(Serial, "application/json");
 httpReply.send("{\"All your base are belong to us!\"}");
 ```
 
+Documentation
+-------------
+
+Documentation available in the [ArduinoHttpServer Github wiki](https://github.com/QuickSander/ArduinoHttpServer/wiki)
+
 Characteristics
 ---------------
 * HTTP parser with protocol validation.

library.json

@@ -1,7 +1,7 @@
 {
     "name": "ArduinoHttpServer",
     "keywords": "http, get, post, delete, request, reply, protocol, web",
-    "description": "Server side minimalistic HTTP protocol implementation.",
+    "description": "Server side minimalistic HTTP protocol implementation that leaves you in control",
     "authors": {
       "name": "Sander van Woensel",
       "url": "http://quicksander.net"

src/internals/HttpVersion.cpp

@@ -0,0 +1,47 @@
+//
+//! \file
+//  ArduinoHttpServer
+//
+//  Created by Sander van Woensel on 20-01-16.
+//  Copyright (c) 2016 Sander van Woensel. All rights reserved.
+//
+//! HTTP version
+
+#include "HttpVersion.hpp"
+#include "Debug.h"
+
+const char ArduinoHttpServer::HttpVersion::SEPARATOR = '.';
+
+ArduinoHttpServer::HttpVersion::HttpVersion() :
+   m_major(0),
+   m_minor(0)
+{
+}
+
+ArduinoHttpServer::HttpVersion::HttpVersion(const String& version) :
+   m_major(0),
+   m_minor(0)
+{
+   int dotIndex(version.lastIndexOf(SEPARATOR));
+
+   // Cast might possibly invalidate version data when versions become bigger then 255.
+   m_major = static_cast<unsigned char>( version.substring(0, dotIndex).toInt() );
+   m_minor = static_cast<unsigned char>( version.substring(dotIndex+1, version.length()).toInt() );
+}
+
+ArduinoHttpServer::HttpVersion& ArduinoHttpServer::HttpVersion::operator=(const HttpVersion& rhs)
+{
+   m_major = rhs.m_major;
+   m_minor = rhs.m_minor;
+
+   return *this;
+}
+
+String ArduinoHttpServer::HttpVersion::toString()
+{
+   String version(m_major);
+   version += SEPARATOR;
+   version += m_minor;
+
+   return version;
+}

src/internals/HttpVersion.hpp

@@ -0,0 +1,42 @@
+//
+//! \file
+//  ArduinoHttpServer
+//
+//  Created by Sander van Woensel on 20-01-16.
+//  Copyright (c) 2016 Sander van Woensel. All rights reserved.
+//
+//! An HTTP version
+
+#ifndef __ArduinoHttpServer__HttpVersion__
+#define __ArduinoHttpServer__HttpVersion__
+
+#include "Arduino.h"
+
+namespace ArduinoHttpServer
+{
+
+//! A single HTTP field.
+class HttpVersion
+{
+
+public:
+   HttpVersion(const String& version);
+   HttpVersion();
+
+   HttpVersion& operator=(const HttpVersion& rhs);
+
+   String toString();
+   inline unsigned char getMajor() const { return m_major; }
+   inline unsigned char getMinor() const { return m_minor; }
+
+private:
+   static const char SEPARATOR;
+
+   unsigned char m_major;
+   unsigned char m_minor;
+
+};
+
+}
+
+#endif

src/internals/StreamHttpReply.hpp

@@ -63,12 +63,7 @@ class StreamHttpReply: public AbstractStreamHttpReply
 {
 public:
     StreamHttpReply(Stream& stream, const String& contentType);
-
     virtual void send(const String& data) { AbstractStreamHttpReply::send(data, "OK"); };
-
-
-protected:
-
 };
 
 

src/internals/StreamHttpRequest.hpp

@@ -12,15 +12,18 @@
 
 #include "HttpResource.hpp"
 #include "HttpField.hpp"
+#include "HttpVersion.hpp"
 #include "Debug.h"
 
 #include <Arduino.h>
 
 namespace ArduinoHttpServer
 {
 
-
-   enum MethodEnum {MethodInvalid, MethodGet, MethodPut, MethodPost, MethodHead};
+enum MethodEnum
+{
+   MethodInvalid, MethodGet, MethodPut, MethodPost, MethodHead
+};
 
 
 //------------------------------------------------------------------------------
@@ -41,7 +44,7 @@ class StreamHttpRequest
     // Header retrieval methods.
     inline const ArduinoHttpServer::HttpResource& getResource() const { return m_resource; };
     inline const MethodEnum getMethod() const { return m_method; };
-    inline const String& getVersion() const { return m_version; };
+    inline const ArduinoHttpServer::HttpVersion& getVersion() const { return m_version; };
 
     // Field retrieval methods.
     inline const String& getContentType() const { return m_contentTypeField.getValueAsString(); };
@@ -81,7 +84,7 @@ class StreamHttpRequest
    char m_body[MAX_BODY_SIZE];
    MethodEnum m_method;
    ArduinoHttpServer::HttpResource m_resource;
-   String m_version;
+   ArduinoHttpServer::HttpVersion m_version;
    ArduinoHttpServer::HttpField m_contentTypeField;
    ArduinoHttpServer::HttpField m_contentLengthField;
 
@@ -248,7 +251,7 @@ void ArduinoHttpServer::StreamHttpRequest<MAX_BODY_LENGTH>::parseVersion()
     // String returns unsigned int for length.
     if (static_cast<unsigned int>(slashPosition) < version.length() && slashPosition > 0)
     {
-        m_version = version.substring(slashPosition);
+        m_version = HttpVersion(version.substring(slashPosition));
     }
     else
     {