start switching DataExchangeFormat::reader/writer code to using SharedByValue - fixing GetRep's to reutnr bare ptr/ref instead of shared_ptr to avoid that dependnecy (and simpler to use - moves asserts)
minor cleanups - and lose uneeded #include