Changeset ef59bc73bec75f7ab931c023663f3d3e50f569c4

Show
Ignore:
Timestamp:
03/14/08 08:05:31 (6 years ago)
Author:
Theo Schlossnagle <jesus@omniti.com>
git-committer:
Theo Schlossnagle <jesus@omniti.com> 1205481931 +0000
git-parent:

[eacf8d5134454850a8d9391db1790ce0df6d6f05]

git-author:
Theo Schlossnagle <jesus@omniti.com> 1205481931 +0000
Message:

cleanup some makefiles and code -- make it easier to see warnings

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • src/Makefile.in

    r257eda6 ref59bc7  
    4848 
    4949noitd:  make-subdirs $(NOIT_OBJS) 
    50         $(CC) -o $@ $(NOIT_OBJS) \ 
     50        @$(CC) -o $@ $(NOIT_OBJS) \ 
    5151                $(LDFLAGS) \ 
    5252                -Leventer -leventer \ 
     
    5555                -Lnoitedit -lnoitedit \ 
    5656                $(LIBS) 
     57        @echo "- linking $@" 
    5758 
    5859stratcond:      make-subdirs $(STRATCON_OBJS) 
    59         $(CC) -o $@ $(STRATCON_OBJS) \ 
     60        @$(CC) -o $@ $(STRATCON_OBJS) \ 
    6061                $(LDFLAGS) \ 
    6162                -Leventer -leventer \ 
     
    6465                -Lnoitedit -lnoitedit \ 
    6566                $(LIBS) $(PGLIBS) 
     67        @echo "- linking $@" 
    6668 
    6769.c.o: 
    68         $(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     70        @$(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     71        @echo "- compiling $<" 
    6972 
    7073noit_tokenizer.c:       noit_tokenizer.re 
    71         re2c -o $@ noit_tokenizer.re 
     74        @re2c -o $@ noit_tokenizer.re 
     75        @echo "- re2c $<" 
    7276 
    7377clean-subdirs: 
  • src/eventer/Makefile.in

    rb7ec807 ref59bc7  
    1919 
    2020libeventer.a:   $(OBJS) 
    21         rm -f $@ 
    22         $(AR) cq $@ $(OBJS) 
    23         $(RANLIB) $@ 
     21        @rm -f $@ 
     22        @$(AR) cq $@ $(OBJS) 
     23        @echo "- archiving $@" 
     24        @$(RANLIB) $@ 
    2425 
    2526.c.o: 
    26         $(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     27        @$(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     28        @echo "- compiling $<" 
    2729 
    2830clean: 
  • src/eventer/eventer.c

    r171d8ed ref59bc7  
    1010} 
    1111 
    12 int eventer_timecompare(void *av, void *bv) { 
     12int eventer_timecompare(const void *av, const void *bv) { 
    1313  /* Herein we avoid equality.  This function is only used as a comparator 
    1414   * for a heap of timed events.  If they are equal, b is considered less 
    1515   * just to maintain an order (despite it not being stable). 
    1616   */ 
    17   eventer_t a = (eventer_t)av; 
    18   eventer_t b = (eventer_t)bv; 
     17  const eventer_t a = (eventer_t)av; 
     18  const eventer_t b = (eventer_t)bv; 
    1919  if(a->whence.tv_sec < b->whence.tv_sec) return -1; 
    2020  if(a->whence.tv_sec == b->whence.tv_sec && 
  • src/eventer/eventer.h

    rb5f532b ref59bc7  
    6161API_EXPORT(eventer_t) eventer_alloc(); 
    6262API_EXPORT(void)      eventer_free(eventer_t); 
    63 API_EXPORT(int)       eventer_timecompare(void *a, void *b); 
     63API_EXPORT(int)       eventer_timecompare(const void *a, const void *b); 
    6464API_EXPORT(int)       eventer_name_callback(const char *name, eventer_func_t f); 
    6565API_EXPORT(const char *) 
  • src/modules/Makefile.in

    r4330e3d ref59bc7  
    2121 
    2222http.@MODULEEXT@:       http.lo 
    23         $(MODULELD) $(LDFLAGS) -o $@ http.lo @SERFLIBS@ ../serf/.libs/libserf-0.a -lz -lssl -lcrypto 
     23        @$(MODULELD) $(LDFLAGS) -o $@ http.lo @SERFLIBS@ ../serf/.libs/libserf-0.a -lz -lssl -lcrypto 
     24        @echo "- linking $@" 
    2425 
    2526http.lo:        http.c 
    26         $(CC) $(CPPFLAGS) $(SHCFLAGS) -I$(top_srcdir)/src/serf @SERFCFLAGS@ -c http.c -o $@ 
     27        @$(CC) $(CPPFLAGS) $(SHCFLAGS) -I$(top_srcdir)/src/serf @SERFCFLAGS@ -c http.c -o $@ 
     28        @echo "- compiling $<" 
    2729 
    2830snmp.@MODULEEXT@:       snmp.lo 
    29         $(MODULELD) $(LDFLAGS) -o $@ snmp.lo -lnetsnmp 
     31        @$(MODULELD) $(LDFLAGS) -o $@ snmp.lo -lnetsnmp 
     32        @echo "- linking $@" 
    3033 
    3134ssh2.@MODULEEXT@:       ssh2.lo 
    32         $(MODULELD) $(LDFLAGS) -o $@ ssh2.lo -lssh2 
     35        @$(MODULELD) $(LDFLAGS) -o $@ ssh2.lo -lssh2 
     36        @echo "- linking $@" 
    3337 
    3438.lo.@MODULEEXT@: 
    35         $(MODULELD) $(LDFLAGS) -o $@ $< 
     39        @$(MODULELD) $(LDFLAGS) -o $@ $< 
     40        @echo "- linking $@" 
    3641 
    3742.c.lo: 
    38         $(CC) $(CPPFLAGS) $(SHCFLAGS) -c $< -o $@ 
     43        @$(CC) $(CPPFLAGS) $(SHCFLAGS) -c $< -o $@ 
     44        @echo "- compiling $<" 
    3945 
    4046clean: 
  • src/noit_check.c

    reacf8d5 ref59bc7  
    5555  return "???"; 
    5656} 
    57 static int __check_name_compare(void *a, void *b) { 
    58   noit_check_t *ac = a; 
    59   noit_check_t *bc = b; 
     57static int __check_name_compare(const void *a, const void *b) { 
     58  const noit_check_t *ac = a; 
     59  const noit_check_t *bc = b; 
    6060  int rv; 
    6161  if((rv = strcmp(ac->target, bc->target)) != 0) return rv; 
  • src/noitedit/Makefile.in

    r42632e7 ref59bc7  
    3131 
    3232libnoitedit.a:   $(FABHEADERS) $(OBJS) 
    33         rm -f $@ 
    34         $(AR) cq $@ $(OBJS) 
    35         $(RANLIB) $@ 
     33        @rm -f $@ 
     34        @$(AR) cq $@ $(OBJS) 
     35        @echo "- archiving $@" 
     36        @$(RANLIB) $@ 
    3637 
    3738.c.o:   vi.h emacs.h common.h fcns.h help.h 
    38         $(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     39        @$(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     40        @echo "- compiling $<" 
    3941 
    4042vi.h: vi.c makelist 
    41         sh ./makelist -h vi.c > $@ 
     43        @sh ./makelist -h vi.c > $@ 
     44        @echo "- makelist $@" 
    4245 
    4346emacs.h: emacs.c makelist 
    44         sh ./makelist -h emacs.c > $@ 
     47        @sh ./makelist -h emacs.c > $@ 
     48        @echo "- makelist $@" 
    4549 
    4650common.h: common.c makelist 
    47         sh ./makelist -h common.c > $@ 
     51        @sh ./makelist -h common.c > $@ 
     52        @echo "- makelist $@" 
    4853 
    4954fcns.h: vi.h emacs.h common.h makelist 
    50         sh ./makelist -fh vi.h emacs.h common.h > $@ 
     55        @sh ./makelist -fh vi.h emacs.h common.h > $@ 
     56        @echo "- makelist $@" 
    5157 
    5258fcns.c: fcns.h 
    53         sh ./makelist -fc vi.h emacs.h common.h > $@ 
     59        @sh ./makelist -fc vi.h emacs.h common.h > $@ 
     60        @echo "- makelist $@" 
    5461 
    5562help.c: vi.c emacs.c common.c makelist 
    56         sh ./makelist -bc vi.c emacs.c common.c > $@ 
     63        @sh ./makelist -bc vi.c emacs.c common.c > $@ 
     64        @echo "- makelist $@" 
    5765 
    5866help.h: vi.c emacs.c common.c makelist 
    59         sh ./makelist -bh vi.c emacs.c common.c > $@ 
     67        @sh ./makelist -bh vi.c emacs.c common.c > $@ 
     68        @echo "- makelist $@" 
    6069 
    6170editline.c: vi.c emacs.c common.c makelist 
    62         sh ./makelist -e vi.c emacs.c common.c > $@ 
     71        @sh ./makelist -e vi.c emacs.c common.c > $@ 
     72        @echo "- makelist $@" 
    6373 
    6474clean: 
  • src/utils/Makefile.in

    r9a02263 ref59bc7  
    1515 
    1616libnoit_utils.a:        $(OBJS) 
    17         rm -f $@ 
    18         $(AR) cq $@ $(OBJS) 
    19         $(RANLIB) $@ 
     17        @rm -f $@ 
     18        @$(AR) cq $@ $(OBJS) 
     19        @echo "- archiving $@" 
     20        @$(RANLIB) $@ 
    2021 
    2122.c.o: 
    22         $(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     23        @$(CC) $(CPPFLAGS) $(CFLAGS) -c $< 
     24        @echo "- compiling $<" 
    2325 
    2426clean: 
  • src/utils/noit_skiplist.c

    r4cdbcdb ref59bc7  
    1818#include <stdio.h> 
    1919#include <stdlib.h> 
     20#include <string.h> 
    2021#include <assert.h> 
    2122 
     
    5051 
    5152void noit_skiplisti_init(noit_skiplist *sl) { 
    52   sl->compare = (noit_skiplist_comparator_t)NULL; 
    53   sl->comparek = (noit_skiplist_comparator_t)NULL; 
    54   sl->height=0; 
    55   sl->preheight=0; 
    56   sl->size=0; 
    57   sl->top = NULL; 
    58   sl->bottom = NULL; 
    59   sl->index = NULL; 
    60 
    61  
    62 static int indexing_comp(const void *a, const void *b) { 
    63   assert(a); 
    64   assert(b); 
    65   return (void *)(((noit_skiplist *)a)->compare)>(void *)(((noit_skiplist *)b)->compare); 
    66 
    67 static int indexing_compk(const void *a, const void *b) { 
    68   assert(b); 
    69   return a>(void *)(((noit_skiplist *)b)->compare); 
     53  memset(sl, 0, sizeof(*sl)); 
     54
     55 
     56static int indexing_comp(const void *av, const void *bv) { 
     57  const noit_skiplist *a = av; 
     58  const noit_skiplist *b = bv; 
     59  return (void *)(a->compare)>(void *)(b->compare); 
     60
     61static int indexing_compk(const void *a, const void *bv) { 
     62  const noit_skiplist *b = bv; 
     63  return a>(void *)(b->compare); 
    7064} 
    7165 
     
    7872 
    7973void noit_skiplist_set_compare(noit_skiplist *sl, 
    80                    noit_skiplist_comparator_t comp, 
    81                    noit_skiplist_comparator_t compk) { 
     74                               noit_skiplist_comparator_t comp, 
     75                              noit_skiplist_comparator_t compk) { 
    8276  if(sl->compare && sl->comparek) { 
    8377    noit_skiplist_add_index(sl, comp, compk); 
     
    8983 
    9084void noit_skiplist_add_index(noit_skiplist *sl, 
    91                  noit_skiplist_comparator_t comp, 
    92                  noit_skiplist_comparator_t compk) { 
     85                             noit_skiplist_comparator_t comp, 
     86                             noit_skiplist_comparator_t compk) { 
    9387  noit_skiplist_node *m; 
    9488  noit_skiplist *ni; 
     
    232226  if(!sl->top) { 
    233227    sl->height = 1; 
    234     sl->topend = sl->bottomend = sl->top = sl->bottom =  
    235       (noit_skiplist_node *)malloc(sizeof(noit_skiplist_node)); 
    236     assert(sl->top); 
    237     sl->top->next = (noit_skiplist_node *) NULL; 
    238     sl->top->data = (noit_skiplist_node *) NULL; 
    239     sl->top->prev =(noit_skiplist_node *) NULL; 
    240         sl->top->up = (noit_skiplist_node *) NULL; 
    241     sl->top->down = (noit_skiplist_node *) NULL; 
    242         sl->top->nextindex=  (noit_skiplist_node *) NULL; 
    243     sl->top->previndex = (noit_skiplist_node *) NULL; 
     228    sl->top = sl->bottom =  
     229      calloc(1, sizeof(noit_skiplist_node)); 
    244230    sl->top->sl = sl; 
    245231  } 
     
    252238  /* Let us make sure that our tree is a least that tall (grow if necessary)*/ 
    253239  for(;sl->height<nh;sl->height++) { 
    254     sl->top->up = 
    255       (noit_skiplist_node *)malloc(sizeof(noit_skiplist_node)); 
    256     assert(sl->top->up); 
     240    sl->top->up = (noit_skiplist_node *)calloc(1, sizeof(noit_skiplist_node)); 
    257241    sl->top->up->down = sl->top; 
    258     sl->top = sl->topend = sl->top->up; 
    259     sl->top->prev = sl->top->next = sl->top->nextindex = 
    260       sl->top->previndex = sl->top->up = NULL; 
    261     sl->top->data = NULL; 
     242    sl->top = sl->top->up; 
    262243    sl->top->sl = sl; 
    263244  } 
     
    266247  /* Keep a stack to pop back through for insertion */ 
    267248  m = sl->top; 
    268   stack = (noit_skiplist_node **)malloc(sizeof(noit_skiplist_node *)*(nh)); 
     249  stack = (noit_skiplist_node **)alloca(sizeof(noit_skiplist_node *)*(nh)); 
    269250  stacki=0; 
    270251  while(m) { 
     
    272253    if(m->next) compared=comp(data, m->next->data); 
    273254    if(compared == 0) { 
    274       free(stack); 
    275255      return 0; 
    276256    } 
    277     if((m->next == NULL) || (compared<0)) { 
     257    if(compared<0) { 
    278258      if(ch<=nh) { 
    279259        /* push on stack */ 
     
    290270  for(;stacki>0;stacki--) { 
    291271    m = stack[stacki-1]; 
    292     tmp = (noit_skiplist_node *)malloc(sizeof(noit_skiplist_node)); 
     272    tmp = calloc(1, sizeof(*tmp)); 
    293273    tmp->next = m->next; 
    294274    if(m->next) m->next->prev=tmp; 
    295275    tmp->prev = m; 
    296     tmp->up = NULL; 
    297     tmp->nextindex = tmp->previndex = NULL; 
    298276    tmp->down = p; 
    299277    if(p) p->up=tmp; 
     
    305283    p = tmp; 
    306284  } 
    307   free(stack); 
    308285  if(sl->index != NULL) { 
    309286    /* this is a external insertion, we must insert into each index as well */ 
     
    345322    p = sl->top; 
    346323    sl->top = sl->top->down; /* Move top down one */ 
    347     if(sl->top) sl->top->up = NULL;      /* Make it think its the top */ 
     324    if(sl->top) sl->top->up = NULL; /* Make it think its the top */ 
    348325    free(p); 
    349326    sl->height--; 
  • src/utils/noit_skiplist.h

    r4cdbcdb ref59bc7  
    4040  struct _noit_skiplist_node *top; 
    4141  struct _noit_skiplist_node *bottom; 
    42   /* These two are needed for appending */ 
    43   struct _noit_skiplist_node *topend; 
    44   struct _noit_skiplist_node *bottomend; 
    4542  struct _iskiplist *index; 
    4643} noit_skiplist;