On Fri, Mar 8, 2013 at 3:36 AM, Derick Rethans <notifications@github.com> wrote:
>
> In mcon/manager.c:
>
> >  	if (manager->connections) {
> >  		/* Does this recursively for all cons */
> >  		destroy_manager_item(manager, manager->connections);
> >  	}
> > +	for(i=0; i<manager->blacklist_count; i++) {
> > +		free(manager->blacklist[i]);
>
> On Thu, 7 Mar 2013, Hannes Magnusson wrote: > if (manager->connections) {
> > /* Does this recursively for all cons */ > destroy_manager_item(manager,
> manager->connections); > } > + for(i=0; i<manager->blacklist_count; i++) { >
> + free(manager->blacklist[i]); fixed
> Not quite :-) + for(i = 0; i < manager->blacklist_count; i++) { ^ +- space
> missing :-)
Unsure where you are looking but it has already been fixed. There are
several of the "wrong syntax" still in the codebase:
bson.c:      for (i=0; i<buf-buf_start; i++) {
bson.c:  for (i=0; i<len; i++) {
collection.c:  for (i=0; i<key_len; i++) {
mongo_types.c:    for (i=0; i<12; i++) {
mongo_types.c:    for(i=0;i<12;i++) {
mongo_types.c:  for(i=0; i<12; i++) {
mongo_types.c:  for (i=0; i<4; i++) {