Skip to content

EEPROMEx new features request #13

@ievgen-kapinos

Description

@ievgen-kapinos

Hi, I really enjoy with your read/write templates. But when I tried to reuse your library I stuck with many restrictions:

  1. I need to change "cpp" source to disable #define _EEPROMEX_DEBUG. IMHO better way to use #ifndef _EEPROMEX_DEBUG true. It will allow disable debugging and use unchanged library
  2. I tried too inherit your class and reuse most of code with my external memory on I2C AT24C32 chip. But:
  • hardware eeprom_read... calls are spreaded over whole code
  • readByte(), writeByte(), isWriteOk() and isReady() are not virtual
  1. I've application witch work during long period. I want to check _memSize, but not _allowedWrites at the same time
  2. In templates wrong check for last byte . For example isWriteOk(address+sizeof(value)) should be changed to isWriteOk(address+sizeof(value)-1)

I hope my contribution useful.
Thanks,
Evgeny

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions