X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=src%2Fcommon%2Fdynamic-buffer.c;h=73195cc2a5586f621314a9b23b55948edec55c40;hb=093627be892001471417cd94ab307f14377a032a;hp=9ed16514fe64fa84dc735f15de862ca662a04fef;hpb=5b1dd5fb3a3ad3fa972bba3a1916012c018727ff;p=lttng-tools.git

diff --git a/src/common/dynamic-buffer.c b/src/common/dynamic-buffer.c
index 9ed16514f..73195cc2a 100644
--- a/src/common/dynamic-buffer.c
+++ b/src/common/dynamic-buffer.c
@@ -16,10 +16,13 @@
  */
 
 #include <common/dynamic-buffer.h>
-#include <common/macros.h>
 #include <common/utils.h>
 #include <assert.h>
 
+/*
+ * Round to (upper) power of two, val is returned if it already is a power of
+ * two.
+ */
 static
 size_t round_to_power_of_2(size_t val)
 {
@@ -34,12 +37,14 @@ size_t round_to_power_of_2(size_t val)
 	return rounded;
 }
 
+LTTNG_HIDDEN
 void lttng_dynamic_buffer_init(struct lttng_dynamic_buffer *buffer)
 {
 	assert(buffer);
 	memset(buffer, 0, sizeof(*buffer));
 }
 
+LTTNG_HIDDEN
 int lttng_dynamic_buffer_append(struct lttng_dynamic_buffer *buffer,
 		const void *buf, size_t len)
 {
@@ -55,11 +60,11 @@ int lttng_dynamic_buffer_append(struct lttng_dynamic_buffer *buffer,
 		goto end;
 	}
 
-	assert(buffer->capacity >= buffer->size);
-	if (buffer->capacity < (len + buffer->size)) {
+	assert(buffer->_capacity >= buffer->size);
+	if (buffer->_capacity < (len + buffer->size)) {
 		ret = lttng_dynamic_buffer_set_capacity(buffer,
-				buffer->capacity +
-				(len - (buffer->capacity - buffer->size)));
+				buffer->_capacity +
+				(len - (buffer->_capacity - buffer->size)));
 		if (ret) {
 			goto end;
 		}
@@ -71,6 +76,7 @@ end:
 	return ret;
 }
 
+LTTNG_HIDDEN
 int lttng_dynamic_buffer_append_buffer(struct lttng_dynamic_buffer *dst_buffer,
 		struct lttng_dynamic_buffer *src_buffer)
 {
@@ -87,6 +93,7 @@ end:
 	return ret;
 }
 
+LTTNG_HIDDEN
 int lttng_dynamic_buffer_set_size(struct lttng_dynamic_buffer *buffer,
 		size_t new_size)
 {
@@ -100,9 +107,9 @@ int lttng_dynamic_buffer_set_size(struct lttng_dynamic_buffer *buffer,
 		goto end;
 	}
 
-	if (new_size > buffer->capacity) {
+	if (new_size > buffer->_capacity) {
 		size_t original_size = buffer->size;
-		size_t original_capacity = buffer->capacity;
+		size_t original_capacity = buffer->_capacity;
 
 		ret = lttng_dynamic_buffer_set_capacity(buffer, new_size);
 		if (ret) {
@@ -135,12 +142,14 @@ end:
 	return ret;
 }
 
+LTTNG_HIDDEN
 int lttng_dynamic_buffer_set_capacity(struct lttng_dynamic_buffer *buffer,
 		size_t demanded_capacity)
 {
 	int ret = 0;
 	void *new_buf;
-	size_t new_capacity = round_to_power_of_2(demanded_capacity);
+	size_t new_capacity = demanded_capacity ?
+			round_to_power_of_2(demanded_capacity) : 0;
 
 	if (!buffer || demanded_capacity < buffer->size) {
 		/*
@@ -151,7 +160,7 @@ int lttng_dynamic_buffer_set_capacity(struct lttng_dynamic_buffer *buffer,
 		goto end;
 	}
 
-	if (new_capacity == buffer->capacity) {
+	if (new_capacity == buffer->_capacity) {
 		goto end;
 	}
 
@@ -162,18 +171,19 @@ int lttng_dynamic_buffer_set_capacity(struct lttng_dynamic_buffer *buffer,
 		goto end;
 	}
 	buffer->data = new_buf;
-	buffer->capacity = new_capacity;
+	buffer->_capacity = new_capacity;
 end:
 	return ret;
 }
 
 /* Release any memory used by the dynamic buffer. */
+LTTNG_HIDDEN
 void lttng_dynamic_buffer_reset(struct lttng_dynamic_buffer *buffer)
 {
 	if (!buffer) {
 		return;
 	}
 	buffer->size = 0;
-	buffer->capacity = 0;
+	buffer->_capacity = 0;
 	free(buffer->data);
 }