in BoxController.cpp == operator. Comments are necessary to highlight the assumptions, made when defining the operator.

Hate to use features I do not understand to the code I have not written but it seems does not do any harm

to the MDBoxFlatTree methors (boxID !=boxNum?

(though in very ugly way)

(should be unit tests for that particular case)

It is pain to have BC shared pointer in Nexus loader. It is also question if it needs access to box controller at all. This is why it is currently accessed through const pointer

It may be not very efficient file mode as one needs to identify the if the buffer for SaveMD make file based is defined correctly.

only MDBoxFlatTree may still need to be modified for this ticket.

Temporary Reverted box controller to with control of boxes to change to vanilla box controller and disabling BC copy constructor, as it is too complex for file based workspaces.

and enabled number of other small things related to file-based boxes

Thisgs to note:
 * C2450 warnings have been suppressed when including Poco headers
   that inherit from std::stream
 * Boost date_time no longer accepts just a date for an ISO date/time
   string

(and leaving most commits for this ticket on private branch except the special one)

hopefully reasonable function, which decreases critical section and allow to claim range of ID-s in one call to it.

Conflicts:

	Code/Mantid/Framework/API/inc/MantidAPI/BoxController.h
	Code/Mantid/Framework/MDEvents/src/MDGridBox.cpp

Fixes #3349: XML box controller. Fix test on some platforms. Refs #3316: Loading/saving box controller to NXS.

Fixes #3351: test for VectorHelper and some refinement of that method to split a comma-delimited string. Refs #3349: BoxController XML serialize/deserialize.