Menu

#107 [Q] merge AEOutputStream, AEFileOutputStream, AEDataFile

open
nobody
None
normal
jAER
None
All
All
2015-04-14
2015-04-14
No

Should we merge these files:
AEOutputStream, AEFileOutputStream, AEDataFile

as they are never used separately, I think it would be more clear to merge them (eg into the AEOutputStream).
What do you think?

Discussion

  • Luca Longinotti

    Luca Longinotti - 2015-04-14

    I'm against merging AEOutputStream and AEFileOutputStream. Separation is not only about usage, but about logical separation into abstract entities too. From that point of view, it makes perfect sense to separate the generic output stream from it's current file implementation.
    I don't have strong feelings about merging AEDataFile into AEFileOutputStream, especially as it's an interface that doesn't define any interface at all, just a few constants. I don't think anything external depends on this, but I'll let Tobi have the final comment on this.

     
  • Marek Otahal, CTU Prague

    I agree on OutputStream and FileOutputStream;

    As part of https://sourceforge.net/p/jaer/tickets/110/, maybe the AEDataFile could become real interface (abstract void write()), and the re-implementations would redefine the constants & write() (writeInt()/writeByte()) as these are coupled to the output-format?

     

Log in to post a comment.