search

bakerstu / openmrn

OpenMRN (Open Model Railroad Network)
BSD 2-Clause "Simplified" License
57 stars 28 forks source link

Incorrect NMRAnetNode.hxx, NMRAnetMessageID.hxx, NMRAnetIf.hxx in Stream.[hc]xx #48

Open evili opened 7 years ago

evili commented 7 years ago

Hi,

There are still three includes that reference obsoleted files. In openlcb/Stream.cxx openlcb/Stream.hxx you can still find the following:

` $ grep -n NMRAnet src/openlcb/* | grep include

src/openlcb/Stream.cxx:35:#include "openlcb/NMRAnetNode.hxx"

src/openlcb/Stream.cxx:36:#include "openlcb/NMRAnetMessageID.hxx"

src/openlcb/Stream.hxx:37:#include "openlcb/NMRAnetIf.hxx"

` These files are missing from the repo since 2014 at commits:

316e43f08

balazsracz commented 7 years ago

Yeah, Stream.hxx and Stream.cxx is not functional at the moment -- in fact they have never been integrated into the stack's asynchronous execution infrastructure. It probably doesn't reflect the current draft Standard for streams either.

We should probably dust it off and actually implement streams handling; it would also significantly reduce the complexity of the BootloaderClient, which currently does the expected stuff barefoot without help from the stack.

Do you have a need / use-case of streams?

Balazs

evili commented 7 years ago

Hi,

I do not use Streams. I was just trying to compile with eclipse.

Thanks, Evili