Idfxt

This pull request is for the sake of conversation, not necessarily with the intent to merge.  yet. 
On Wednesday, December 17, 2014 01:01:43 PM Kyle Benne wrote:
> > +
> > +#include <string>
> > +
> > +extern "C"
> > +{
> > +#include "../../third_party/cJSON/cJSON.h"
> > +}
> > +
> > +namespace idfx {
> > +
> > +class JSONDataInterface
> > +{
> > +public:
> > +    JSONDataInterface() : JSONDataInterface(""){};
> > +    JSONDataInterface(const JSONDataInterface&) = delete;
> > +    JSONDataInterface(std::string json_schema);
> 
> You should pass the json_schema string by const reference.
> 

right.  I'd just plainly used it the way cJSON had.  I believe I can address 
that.

> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/NREL/EnergyPlus/pull/4636/files#r22003778