Print

Print


I'm making use of the new XrdClient code and have found some 
'interesting' items.  The two most pressing at the moment are

1) the definition of XrdClient::Read(...)
	 int          Read(const void *buf, long long offset, int len);

the first variable should be changed to 'void *' since one should not 
be writing to a const buffer.

2) The includes in XrdClient.hh (and probably other header files in 
that package) do not include the prefix "XrdClient/" when referring to 
other header files in this same area.  This makes using these header 
files a mess.  It would be very helpful if this was corrected.

Also, Doxygen style comments in the header (at least in XrdClient.hh) 
would go a long way to aiding its use.

	Chris Jones
	Cornell University