From 50f56f4ed0e16fd9cc0c2932fb4c4ab0bb921194 Mon Sep 17 00:00:00 2001 From: Bernhard Kirchen Date: Sun, 3 Sep 2023 17:27:18 +0200 Subject: [PATCH] VE.Direct: whitespace cleanups remove trailing whitespace and fix mixed indentation. --- .../VeDirectFrameHandler.cpp | 39 +++++++++---------- .../VeDirectFrameHandler.h | 8 ++-- 2 files changed, 23 insertions(+), 24 deletions(-) diff --git a/lib/VeDirectFrameHandler/VeDirectFrameHandler.cpp b/lib/VeDirectFrameHandler/VeDirectFrameHandler.cpp index e225f450c..64ad6c352 100644 --- a/lib/VeDirectFrameHandler/VeDirectFrameHandler.cpp +++ b/lib/VeDirectFrameHandler/VeDirectFrameHandler.cpp @@ -2,23 +2,23 @@ * * Arduino library to read from Victron devices using VE.Direct protocol. * Derived from Victron framehandler reference implementation. - * + * * The MIT License - * + * * Copyright (c) 2019 Victron Energy BV * Portions Copyright (C) 2020 Chris Terwilliger * https://github.com/cterwilliger/VeDirectFrameHandler - * + * * Permission is hereby granted, free of charge, to any person obtaining a copy * of this software and associated documentation files (the "Software"), to deal * in the Software without restriction, including without limitation the rights * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell * copies of the Software, and to permit persons to whom the Software is * furnished to do so, subject to the following conditions: - * + * * The above copyright notice and this permission notice shall be included in * all copies or substantial portions of the Software. - * + * * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE @@ -26,13 +26,13 @@ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN * THE SOFTWARE. - * + * * 2020.05.05 - 0.2 - initial release * 2020.06.21 - 0.2 - add MIT license, no code changes * 2020.08.20 - 0.3 - corrected #include reference - * + * */ - + #include #include "VeDirectFrameHandler.h" @@ -54,8 +54,8 @@ HardwareSerial VedirectSerial(1); VeDirectFrameHandler VeDirect; class Silent : public Print { - public: - size_t write(uint8_t c) final { return 0; } + public: + size_t write(uint8_t c) final { return 0; } }; static Silent MessageOutputDummy; @@ -134,7 +134,7 @@ void VeDirectFrameHandler::rxData(uint8_t inbyte) if ( (inbyte == ':') && (_state != CHECKSUM) ) { _prevState = _state; //hex frame can interrupt TEXT - _state = RECORD_HEX; + _state = RECORD_HEX; _hexSize = 0; } if (_state != RECORD_HEX) { @@ -237,7 +237,7 @@ void VeDirectFrameHandler::textRxEvent(char * name, char * value) { else if (strcmp(name, "LOAD") == 0) { if (strcmp(value, "ON") == 0) _tmpFrame.LOAD = true; - else + else _tmpFrame.LOAD = false; } else if (strcmp(name, "CS") == 0) { @@ -282,12 +282,11 @@ void VeDirectFrameHandler::textRxEvent(char * name, char * value) { else if (strcmp(name, "H23") == 0) { _tmpFrame.H23 = atoi(value); } - } /* - * frameEndEvent + * frameEndEvent * This function is called at the end of the received frame. If the checksum is valid, the temp buffer is read line by line. * If the name exists in the public buffer, the new value is copied to the public buffer. If not, a new name/value entry * is created in the public buffer. @@ -314,8 +313,8 @@ void VeDirectFrameHandler::frameEndEvent(bool valid) { } /* - * hexRxEvent - * This function records hex answers or async messages + * hexRxEvent + * This function records hex answers or async messages */ int VeDirectFrameHandler::hexRxEvent(uint8_t inbyte) { int ret=RECORD_HEX; // default - continue recording until end of frame @@ -325,7 +324,7 @@ int VeDirectFrameHandler::hexRxEvent(uint8_t inbyte) { // restore previous state ret=_prevState; break; - + default: _hexSize++; if (_hexSize>=VE_MAX_HEX_LEN) { // oops -buffer overflow - something went wrong, we abort @@ -334,7 +333,7 @@ int VeDirectFrameHandler::hexRxEvent(uint8_t inbyte) { ret=IDLE; } } - + return ret; } @@ -350,7 +349,7 @@ bool VeDirectFrameHandler::isDataValid() { unsigned long VeDirectFrameHandler::getLastUpdate() { - return _lastUpdate; + return _lastUpdate; } /* @@ -556,7 +555,7 @@ String VeDirectFrameHandler::getPidAsString(uint16_t pid) break; case 0XA110: strPID = "SmartSolar MPPT RS 450|100"; - break; + break; case 0XA112: strPID = "BlueSolar MPPT VE.Can 250|70"; break; diff --git a/lib/VeDirectFrameHandler/VeDirectFrameHandler.h b/lib/VeDirectFrameHandler/VeDirectFrameHandler.h index f6ed833ee..6a43bb64b 100644 --- a/lib/VeDirectFrameHandler/VeDirectFrameHandler.h +++ b/lib/VeDirectFrameHandler/VeDirectFrameHandler.h @@ -2,11 +2,11 @@ * * Arduino library to read from Victron devices using VE.Direct protocol. * Derived from Victron framehandler reference implementation. - * + * * 2020.05.05 - 0.2 - initial release * 2021.02.23 - 0.3 - change frameLen to 22 per VE.Direct Protocol version 3.30 * 2022.08.20 - 0.4 - changes for OpenDTU - * + * */ #pragma once @@ -83,7 +83,7 @@ class VeDirectFrameHandler { void loop(); // main loop to read ve.direct data unsigned long getLastUpdate(); // timestamp of last successful frame read bool isDataValid(); // return true if data valid and not outdated - String getPidAsString(uint16_t pid); // product id as string + String getPidAsString(uint16_t pid); // product id as string String getCsAsString(uint8_t cs); // current state as string String getErrAsString(uint8_t err); // errer state as string String getOrAsString(uint32_t offReason); // off reason as string @@ -103,7 +103,7 @@ class VeDirectFrameHandler { bool _verboseLogging; int _state; // current state int _prevState; // previous state - uint8_t _checksum; // checksum value + uint8_t _checksum; // checksum value char * _textPointer; // pointer to the private buffer we're writing to, name or value int _hexSize; // length of hex buffer char _name[VE_MAX_VALUE_LEN]; // buffer for the field name