Skip to content

Commit 7c6c04e

Browse files
committed
fix: cleared warnings (#175)
1 parent 1350fda commit 7c6c04e

File tree

4 files changed

+11
-13
lines changed

4 files changed

+11
-13
lines changed

CHANGELOG.md

+3-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
- Arduino `String` class
66
- C `char *` or `char[]`
77
- Flash string using `F`,`PSTR` or `FPSTR` macros
8+
### Fixes
9+
- [176](https://github.com/tobiasschuerg/InfluxDB-Client-for-Arduino/pull/176) - Cleared all compiler warnings
810

911
## 3.10.0 [2022-01-20]
1012
### Features
@@ -135,4 +137,4 @@
135137
- Richer set of data types for fields and timestamp methods
136138
- Advanced features, such as implicit batching, automatic retrying on server back-pressure and connection failure, along with secured communication over TLS supported for both devices and authentication
137139
- Special characters escaping
138-
- Backward support for original API of V1/V2
140+
- Backward support for original API of V1/V2

src/InfluxDbClient.cpp

+5-8
Original file line numberDiff line numberDiff line change
@@ -681,8 +681,8 @@ InfluxDBClient::BatchStreamer::BatchStreamer(InfluxDBClient::Batch *batch) {
681681
_length = 0;
682682
_pointer = 0;
683683
_linePointer = 0;
684-
for(int i=0;i<_batch->pointer;i++) {
685-
_length += strlen(_batch->buffer[i])+1;
684+
for(uint16_t i=0;i<_batch->pointer;i++) {
685+
_length += strlen(_batch->buffer[i])+1;
686686
}
687687
}
688688

@@ -703,8 +703,8 @@ int InfluxDBClient::BatchStreamer::read(uint8_t* buffer, size_t len) {
703703
size_t InfluxDBClient::BatchStreamer::readBytes(char* buffer, size_t len) {
704704

705705
INFLUXDB_CLIENT_DEBUG("BatchStream::readBytes %d\n", len);
706-
int r=0;
707-
for(int i=0;i<len;i++) {
706+
unsigned int r=0;
707+
for(unsigned int i=0;i<len;i++) {
708708
if(available()) {
709709
buffer[i] = read();
710710
r++;
@@ -740,12 +740,9 @@ int InfluxDBClient::BatchStreamer::peek() {
740740
} else {
741741
r = _batch->buffer[_pointer][_linePointer];
742742
}
743-
// #ifdef INFLUXDB_CLIENT_DEBUG_ENABLE
744-
// Serial.printf_P(PSTR("%c"), r);
745-
// #endif
746743
return r;
747744
}
748745

749-
size_t InfluxDBClient::BatchStreamer::write(uint8_t data) {
746+
size_t InfluxDBClient::BatchStreamer::write(uint8_t) {
750747
return 0;
751748
}

src/query/FluxTypes.cpp

+2-3
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,7 @@ const char *FluxBinaryDataTypeBase64 = "base64Binary";
3838
const char *FluxDatatypeDatetimeRFC3339 = "dateTime:RFC3339";
3939
const char *FluxDatatypeDatetimeRFC3339Nano = "dateTime:RFC3339Nano";
4040

41-
FluxBase::FluxBase(const String &rawValue) {
42-
_rawValue = rawValue;
41+
FluxBase::FluxBase(const String &rawValue):_rawValue(rawValue) {
4342
}
4443

4544
FluxBase::~FluxBase() {
@@ -144,7 +143,7 @@ FluxString::FluxString(const String &rawValue, const char *type):FluxString(rawV
144143

145144
}
146145

147-
FluxString::FluxString(const String &rawValue, const String &value, const char *type):FluxBase(rawValue),value(value),_type(type)
146+
FluxString::FluxString(const String &rawValue, const String &value, const char *type):FluxBase(rawValue),_type(type),value(value)
148147
{
149148

150149
}

src/query/Params.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ QueryParams &QueryParams::operator=(const QueryParams &other) {
5252
}
5353

5454
QueryParams &QueryParams::add(const String &name, float value, int decimalPlaces) {
55-
return add(name, (double)value);
55+
return add(name, (double)value, decimalPlaces);
5656
}
5757

5858
QueryParams &QueryParams::add(const String &name, double value, int decimalPlaces) {

0 commit comments

Comments
 (0)