> BTW I'm curious if you actually needed the + and - suffixes in practice? > I can easily imagine how the = suffix will be useful in the real > world, but + and -... Or maybe just for bus driver testing purpose? Exactly for that. While you can send complex messages with my tool, it should be clear this is mainly for testing/developing. Once you know what you need to do to access a slave, a specfic driver is the better option, be it in userspace or kernel space. Those patterns are easily recognizable when fed into an EEPROM. They can already reveal quite some problems with bus drivers, e.g. off-by-one errors when fetching data, sending STOP etc. > > +} > > + /* let Linux free malloced memory on termination */ > > I don't like this. The memory allocated in i2cbusses.c is freed > explicitly, so it is inconsistent to not free yours. Freeing the memory Well, it is documented :) > explicitly makes the code easier to read and debug as it documents the I disagree about this one. Currently, we have a LOT of error paths when parsing input fails. Getting all the error paths right with freeing memory is usually error-prone (especially with later additions) and will IMO make the code way less readable. Since the lifetime of those buffers ends right before the exit(0), it seems unnecessarily complex to me. However, I need to redo the parsing anyhow. Let's see how it looks like in the next version. If it is easy/readable to free(), I'll do it.
Attachment:
signature.asc
Description: Digital signature
