Can't we make these methods match the same signature and behavior? I understand why Application should handle other features/behaviors but changing the signature/calling convention in this way seems like an unneeded inconsistency. 

On Jul 4, 2013, at 10:17 PM, tschw <notifications@github.com> wrote:

> In interface/src/Application.cpp:
> 
> >      pingPacket[0] = PACKET_HEADER_PING;
> >      
> >      memcpy(&pingPacket[1], &currentTime, sizeof(currentTime));
> > -    NodeList::getInstance()->broadcastToNodes((unsigned char*)pingPacket, 1 + sizeof(currentTime), nodeTypesOfInterest, 3);
> > +    for (int i = 0; i < sizeof(nodeTypesOfInterest) / sizeof(char); ++i) {
> It's calling the function Application::broadcastToNodes (not NodeList::broadcastToNodes)
> that only takes a single type, eventually mutes voxel data when voxels are off and feeds 
> the bandwidth meter (besides passing the request on to NodeList::broadcastToNodes).
> 
> —
> Reply to this email directly or view it on GitHub.