Todo List

Member AISResources::get_resource (const string &primary)
Make this return an empty ResourceVector is no matching resources are found. Clients would not need to call has_resource() which would save some time.

Member Vector::var (unsigned int i)
Is this method thread safe? If 'apartment threding' is used, I think so. But if the library is running in more than one thread, then this is not thread safe.

Member GeoConstraint::reorder_data_longitude_axis (Array &a)
Fix this code so that it works with latitude as the rightmost map

Class AttrTable
Look at refactoring this by splitting it into three classes. Move the struct entry into its own calls (maybe called Attribute?), make AttrTable a child of that class and then make aliases a separate class, also a child of Attribute. Look at the design of the Java code.

Class AttrTable
A problem with this class is that Attr_iter objects cannot be dereferenced to return attributes. Instead they must be passed to methods which require that you have access to the AttrTable object into which they point.03/09/04 jhrg

Class BaseType
We really need a better way to get values out of these types, esp. the Float32, Int16, ..., types. In most cases we know the type, so a type specific method (one that requires a downcast to use) is OK. For example, Byte might have a method dods_byte Byte::value(). Sure you have to downcast from BaseType to Byte in order to use it, but you have to figure out you have a Byte to use Byte::buf2val() anyway, so what's the big deal? Having a method that returns the value would simplify code that reads from data sets to extract meta data (like lat/lon corner points, et c.).

Member BaseType::add_var (BaseType *bt, Part part=nil)
We should get rid of the Part parameter and adopt the convention that the first variable is the Array and all subsequent ones are Maps (when dealing with a Grid, the only time Part matters). This would enable several methods to migrate from Structure, Sequence and Grid to Constructor.

Member BaseType::set_read_p (bool state)
Look at making synthesized variables easier to implement and at making them more integrated into the overall CE evaluation process. Maybe the code that computes the synthesized var's value should be in the that variable's read() method? This might provide a way to get rid of the awkward 'projection functions' by replacing them with real children of BaseType. It would also provide a way to clean up the way the synthesized_p prop intrudes on the read_p prop.

Member Constructor::is_linear ()
Change the name to is_flattenable or something like that. 05/16/03 jhrg

Member DDS::mark (const string &name, bool state)
This should throw an exception on error!!!

Member DDS::mark (const string &name, bool state)
These methods that use the btp_stack to keep track of the path from the top of a dataset to a particular variable can be rewritten to use the parent field instead.

Member DDS::mark (const string &name, bool state)
All the methods that use names to identify variables should have counterparts that take BaseType pointers.

Member DDS::transfer_attributes (DAS *das)
Generalize the code that treats the _dim_? attributes or make is obsolete by fixing the HDF4 server.

Class DODSFilter
Add a test to make sure that the required arguments are given.

We need to rethink the ancillary file/directory stuff. I don't think it's ever been used...

Member DODSFilter::DODSFilter ()
Add methods to provide a way to set all of the parameters this class contains. They can currently only be set using the argc/argv command line parameters.

Member DODSFilter::establish_timeout (FILE *stream) const
When the alarm handler is called, two CRLF pairs are dumped to the stream and then an Error object is sent. No attempt is made to write the 'correct' MIME headers for an Error object. Instead, a savvy client will know that when an exception is thrown during a deserialize operation, it should scan ahead in the input stream for an Error object. Add this, or a sensible variant once libdap++ supports reliable error delivery. Dumb clients will never get the Error object...

Class Grid
Move, in some sense, the _map_vars up to Constructor. Look at using Constructor's _var field for these.

Along the same lines as the previous item, consider removing the Part enum and adopting the convention that the first variable added is the array and any subsequent variables are maps.

Class HTTPCache
Because is_url_in_cache() and is_url_valid() are discrete, an interface that combines the two might be easier to use. Or maybe if is_url_valid() threw a special exception if the entry was missing. Something to help clients deal with URLs that are removed from the cache in between calls to the two methods.

Class HTTPCache
Change the entry locking scheme to distinguish between entries accessed for reading and for writing.

Class HTTPCache
Test in MT software. Is the entry locking scheme good enough? The current software throws an exception if there's an attempt to modify an entry that is locked by another thread. Maybe it should block instead? Maybe we should provide a tests to see if an update would block (one that returns right away and one that blocks). Note: Rob Morris added tests for MT-safety. 02/06/03 jhrg

Class HTTPConnect
Change the way this class returns information so that the headers and the stream (aka FILE pointer) are returned using an object. Design this object so that its destructor closes the stream (this will prevent resource leaks). It will also obviate the need for the (now broken) is_response_present() predicate.

Class HTTPResponse
Maybe refactor so that the header parsing code is hear and not in HTTPConnect?

Member Structure::val2buf (void *val, bool reuse=false)
Make this throw an exception

Class Response
If the code that parses the MIME headers was moved from Connect and HTTPConnect to this class and its children, it would be easier to build a FileConnect class (or maybe the specifics of the connection type could be held in the Response object and HTTPConnect and the to be written FileConnect would not be needed).

Class Sequence
Refactor along with Structure moving methods up into Constructor.

Class Sequence
Add an isEmpty() method which returns true if the Sequence is empty. This should work before and after calling deserialize().

Class Structure
Refactor with Sequence moving methods up into Constructor.

Member libdap::function_grid (int argc, BaseType *argv[], DDS &, const string &dataset)
In order to be used by geogrid() , this code may have to be modified so that the maps and array are not re-read by the serialize() method. It might also be a good idea to change from the '?grid(SST,"10<time<20")' syntax in a URL to '?SST&grid(SST,"10<time<20")' even though it's more verbose in the URL, it would make the function a true 'selection operator' and allow several grids to be returned with selections in one request.

Member prune_spaces
Is this still needed? This function may predate the switch from libwww to libcurl and the latter may not need to have spaces removed.

Generated on Wed Jun 27 12:57:16 2007 for libdap++ by  doxygen 1.4.7