Make default constructor safer

esphome/core/bytebuffer.cpp

@@ -4,9 +4,11 @@
 
 namespace esphome {
 
-ByteBuffer ByteBuffer::create(size_t capacity) {
+ByteBuffer ByteBuffer::create(size_t capacity, Endian endianness) {
   std::vector<uint8_t> data(capacity);
-  return {data};
+  ByteBuffer buffer = {data};
+  buffer.endianness_ = endianness;
+  return buffer;
 }
 
 ByteBuffer ByteBuffer::wrap(const uint8_t *ptr, size_t len) {
@@ -23,46 +25,31 @@ ByteBuffer ByteBuffer::wrap(uint8_t value) {
 }
 
 ByteBuffer ByteBuffer::wrap(uint16_t value, Endian endianness) {
-  ByteBuffer buffer = ByteBuffer::create(2);
+  ByteBuffer buffer = ByteBuffer::create(2, endianness);
-  if (endianness == BIG) {
-    buffer.big_endian();
-  }
   buffer.put_uint16(value);
   return buffer;
 }
 
 ByteBuffer ByteBuffer::wrap(uint32_t value, Endian endianness) {
-  ByteBuffer buffer = ByteBuffer::create(4);
+  ByteBuffer buffer = ByteBuffer::create(4, endianness);
-  if (endianness == BIG) {
-    buffer.big_endian();
-  }
   buffer.put_uint32(value);
   return buffer;
 }
 
 ByteBuffer ByteBuffer::wrap(uint64_t value, Endian endianness) {
-  ByteBuffer buffer = ByteBuffer::create(8);
+  ByteBuffer buffer = ByteBuffer::create(8, endianness);
-  if (endianness == BIG) {
-    buffer.big_endian();
-  }
   buffer.put_uint64(value);
   return buffer;
 }
 
 ByteBuffer ByteBuffer::wrap(float value, Endian endianness) {
-  ByteBuffer buffer = ByteBuffer::create(sizeof(float));
+  ByteBuffer buffer = ByteBuffer::create(sizeof(float), endianness);
-  if (endianness == BIG) {
-    buffer.big_endian();
-  }
   buffer.put_float(value);
   return buffer;
 }
 
 ByteBuffer ByteBuffer::wrap(double value, Endian endianness) {
-  ByteBuffer buffer = ByteBuffer::create(sizeof(double));
+  ByteBuffer buffer = ByteBuffer::create(sizeof(double), endianness);
-  if (endianness == BIG) {
-    buffer.big_endian();
-  }
   buffer.put_double(value);
   return buffer;
 }

esphome/core/bytebuffer.h

@@ -31,11 +31,11 @@ enum Endian { LITTLE, BIG };
 class ByteBuffer {
  public:
   // Default constructor (compatibility with TEMPLATABLE_VALUE)
-  ByteBuffer() = default;
+  ByteBuffer() : ByteBuffer(std::vector<uint8_t>()) {}
   /**
    * Create a new Bytebuffer with the given capacity
    */
-  static ByteBuffer create(size_t capacity);
+  static ByteBuffer create(size_t capacity, Endian endianness = LITTLE);
   /**
    * Wrap an existing vector in a ByteBufffer
    */