mBed online compiler fix + struct example

[pilotak]

No description provided.

[geky]

My only qualm, is that this should probably be dynamically allocated. With BLOCK_SIZE == 32 this is fine, but if we reach a point where we adopt NAND flash, their block sizes can get up into the 4096+ byte range, which runs the risk of stack overflow.

A bit of a nit, but is probably important on the example we show.

[pilotak]

You are right, it should be. But as this is a eeprom branch of block memory library and size of eeprom needs to be set by user, it should be fine.

[geky]

This is a good example to have!

Just a bit of feedback, I'm being a bit picky since this is an example of the front of the repo.

[geky]

If you're changing this file, this should have a bit of a style update:

+ if(i2cee.read(buffer, 0, setting_block_size ) == 0){ // get data into buffer
- if (i2cee.read(buffer, 0, setting_block_size) == 0) { // get data into buffer
      // Convert what we read into struct
      memcpy(&tmp, buffer, sizeof(tmp)); // copy only size of struct not setting_block_size
      printf("\nTemporary struct version: %u, name: %s\n", tmp.version, tmp.name); 
+ } else {
- }
- else {
      printf("Error when reading\n");
  }

[pilotak]

I'm fine with that 👍