You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by an...@apache.org on 2018/11/23 16:16:48 UTC

[mynewt-core] 16/26: apps/bus_test: Remove redundant local variable

This is an automated email from the ASF dual-hosted git repository.

andk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-core.git

commit 14fecdb340f22874ac8856d6afcddc1084746955
Author: Andrzej Kaczmarek <an...@codecoup.pl>
AuthorDate: Thu Nov 15 16:53:00 2018 +0100

    apps/bus_test: Remove redundant local variable
---
 apps/bus_test/bme280_node/src/bme280_node.c | 10 +---------
 1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/apps/bus_test/bme280_node/src/bme280_node.c b/apps/bus_test/bme280_node/src/bme280_node.c
index 120ecc1..8214e51 100644
--- a/apps/bus_test/bme280_node/src/bme280_node.c
+++ b/apps/bus_test/bme280_node/src/bme280_node.c
@@ -41,18 +41,10 @@ struct reg_val {
 static int8_t
 bme280_read(uint8_t dev_id, uint8_t reg_addr, uint8_t *data, uint16_t len)
 {
-    struct reg_val reg;
     int rc;
 
-    assert(len <= sizeof(reg.val));
-
-    reg.addr = reg_addr;
-
     rc = bus_node_simple_write_read_transact(BME280_ODEV(&g_bme280_node),
-                                             &reg.addr, 1, &reg.val[0], len);
-    if (!rc) {
-        memcpy(data, &reg.val[0], len);
-    }
+                                             &reg_addr, 1, data, len);
 
     return rc;
 }