The abstract type we're using is odp_pktio_t so code and tests surrounding that type should be based on that name to make that clear.
On Wed, May 6, 2015 at 4:41 AM, Maxim Uvarov <[email protected]> wrote: > > > On 6 May 2015 at 12:00, Ciprian Barbu <[email protected]> wrote: > >> On Tue, May 5, 2015 at 7:09 PM, Mike Holmes <[email protected]> >> wrote: >> > >> > >> > On 5 May 2015 at 11:55, Maxim Uvarov <[email protected]> wrote: >> >> >> >> Hello, >> >> >> >> I saw in Christophe patch to rename odp_pktio test to odp_packet_io. >> >> Christophe did that because doxygen generated document has packet_io >> name >> >> for this chapter. I think that we need do rename in doxygen than >> rename test >> >> to better patch all odp_pktio prefixes in the code. >> >> >> > >> > I agree we need the name for the API Module to be used consistently for >> the >> > doxygen, tests and examples. >> >> +1 >> At first I was favouring odp_packet_io, but that's too long, perhaps >> we should rename the doxygen group / module like Maxim suggests. But >> if we do this we should rename the API header as well i.e. >> odp_packet_io.h -> odp_pktio.h >> >> > good catch about include files names, shorter names should be more > readable. > > Maxim. > > > >> > >> > Current API Module names for reference when deciding the correct name. >> > >> >> >> >> Any thoughts on this? >> >> >> >> Thank you, >> >> Maxim. >> >> >> >> _______________________________________________ >> >> lng-odp mailing list >> >> [email protected] >> >> https://lists.linaro.org/mailman/listinfo/lng-odp >> > >> > >> > >> > >> > -- >> > Mike Holmes >> > Technical Manager - Linaro Networking Group >> > Linaro.org │ Open source software for ARM SoCs >> > >> > >> > >> > _______________________________________________ >> > lng-odp mailing list >> > [email protected] >> > https://lists.linaro.org/mailman/listinfo/lng-odp >> > >> > > > _______________________________________________ > lng-odp mailing list > [email protected] > https://lists.linaro.org/mailman/listinfo/lng-odp > >
_______________________________________________ lng-odp mailing list [email protected] https://lists.linaro.org/mailman/listinfo/lng-odp
