refs #426 restore legacy behaviour and hdf4 and mxml functionality

this is not a proper fix, as it changes hdf5 behaviour based on
hdf4 or mxml option being enabled at runtime
The purpose was to narrow down the problem. Working legacy code is
a desired side outcome while implementing a better fix.

Author: zjttoefs

bindings/cpp/NeXusFile.cpp

@@ -1101,6 +1101,17 @@ AttrInfo File::getNextAttr() {
   //string & name, int & length, NXnumtype type) {
   char name[NX_MAXNAMELEN];
   int type;
+
+#if defined(WITH_HDF4) || defined(WITH_MXML)
+  int length;
+  NXstatus status = NXgetnextattr(this->m_file_id, name, &length, &type);
+  if (status == NX_OK) {
+    AttrInfo info;
+    info.type = static_cast<NXnumtype>(type);
+    info.length = length;
+    info.name = string(name);
+    return info;
+#else
   int rank;
   int dim[NX_MAXRANK];
   NXstatus status = NXgetnextattra(this->m_file_id, name, &rank, dim, &type);
@@ -1134,6 +1145,7 @@ AttrInfo File::getNextAttr() {
     // TODO - AttrInfo cannot handle more complex ranks/dimensions, we need to throw an error
     std::cerr << "ERROR iterating through attributes found array attribute not understood by this api" << std::endl;
     throw Exception("getNextAttr failed", NX_ERROR);
+#endif
   }
   else if (status == NX_EOD) {
     AttrInfo info;