On Tue, Feb 19, 2013 at 3:31 PM, Katy Huff <notifications@github.com> wrote:

> In src/Core/Utility/EventManager.cpp:
>
> > +    msg = "Name '" + ev->name() + "' with different schema already exists.";
> > +    throw CycInvalidSchemaErr(msg);
> > +  }
> > +
> > +  if (schemas_.find(ev->title()) == schemas_.end()) {
> > +    schemas_[ev->title()] = ev;
> > +  }
> > +
> > +  events_.push_back(ev);
> > +  if (events_.size() >= dump_count_) {
> > +    notifyBacks();
> > +  }
> > +}
> > +
> > +//- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
> > +void EventManager::notifyBacks() {
>
> A picky comment - Maybe this would be better renamed to "notifyBackends,"
> for readability as well as consistency with stuff like "registerBackend."
>
agreed

>  —
> Reply to this email directly or view it on GitHub<https://github.com/cyclus/cyclus/pull/456/files#r3070397>.
>
>