This is the mail archive of the newlib@sourceware.org mailing list for the newlib project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Add XDR support (only built #if cygwin, for now)

On 02/25/2010 12:45 PM, Corinna Vinschen wrote:
Hi Chuck,

thanks for the patch.  I didn't look much into the actual code yet,
but I have a few points.

Neither did I have a look into the code, but some general remarks/comments from the RTEMS side:

- RTEMS has an xdr port of its own (Originally derived from Sun RPC's original code, but heavily modified since).

- XDR in general has a lot of portability issues, esp. wrt. "exotic targets". Have these exotic targets been taken into account?

- XDR is hardly useful for systems without networking support.
Has this consideration been taken into account (E.g. XDR not depending on "networking routines".)

That said, I'd see your XDR works disabled for RTEMS at least for now and/or to be optional (Preferably defaulting to off).

Ralf

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]