[PATCH 2/2] Use compare_strings() instead of compare_hashdata() where applicable
Thomas Guyot-Sionnest
dermoth at aei.ca
Sun Aug 16 00:18:34 CEST 2009
compare_strings() is a frontend for compare_hashdata() where the 2nd compare
value is always NULL.
---
common/comments.c | 8 ++++----
common/statusdata.c | 8 ++++----
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/common/comments.c b/common/comments.c
index 3d15bc5..87356d5 100644
--- a/common/comments.c
+++ b/common/comments.c
@@ -398,8 +398,8 @@ int add_comment_to_hashlist(comment *new_comment){
hashslot=hashfunc(new_comment->host_name,NULL,COMMENT_HASHSLOTS);
lastpointer=NULL;
- for(temp_comment=comment_hashlist[hashslot];temp_comment && compare_hashdata(temp_comment->host_name,NULL,new_comment->host_name,NULL)<0;temp_comment=temp_comment->nexthash){
- if(compare_hashdata(temp_comment->host_name,NULL,new_comment->host_name,NULL)>=0)
+ for(temp_comment=comment_hashlist[hashslot];temp_comment && compare_strings(temp_comment->host_name,new_comment->host_name)<0;temp_comment=temp_comment->nexthash){
+ if(compare_strings(temp_comment->host_name,new_comment->host_name)>=0)
break;
lastpointer=temp_comment;
}
@@ -629,9 +629,9 @@ comment *get_next_comment_by_host(char *host_name, comment *start){
else
temp_comment=start->nexthash;
- for(;temp_comment && compare_hashdata(temp_comment->host_name,NULL,host_name,NULL)<0;temp_comment=temp_comment->nexthash);
+ for(;temp_comment && compare_strings(temp_comment->host_name,host_name)<0;temp_comment=temp_comment->nexthash);
- if(temp_comment && compare_hashdata(temp_comment->host_name,NULL,host_name,NULL)==0)
+ if(temp_comment && compare_strings(temp_comment->host_name,host_name)==0)
return temp_comment;
return NULL;
diff --git a/common/statusdata.c b/common/statusdata.c
index a04c900..ed245f3 100644
--- a/common/statusdata.c
+++ b/common/statusdata.c
@@ -255,10 +255,10 @@ int add_hoststatus_to_hashlist(hoststatus *new_hoststatus){
hashslot=hashfunc(new_hoststatus->host_name,NULL,HOSTSTATUS_HASHSLOTS);
lastpointer=NULL;
- for(temp_hoststatus=hoststatus_hashlist[hashslot];temp_hoststatus && compare_hashdata(temp_hoststatus->host_name,NULL,new_hoststatus->host_name,NULL)<0;temp_hoststatus=temp_hoststatus->nexthash)
+ for(temp_hoststatus=hoststatus_hashlist[hashslot];temp_hoststatus && compare_strings(temp_hoststatus->host_name,new_hoststatus->host_name)<0;temp_hoststatus=temp_hoststatus->nexthash)
lastpointer=temp_hoststatus;
- if(!temp_hoststatus || (compare_hashdata(temp_hoststatus->host_name,NULL,new_hoststatus->host_name,NULL)!=0)){
+ if(!temp_hoststatus || (compare_strings(temp_hoststatus->host_name,new_hoststatus->host_name)!=0)){
if(lastpointer)
lastpointer->nexthash=new_hoststatus;
else
@@ -519,9 +519,9 @@ hoststatus *find_hoststatus(char *host_name){
if(host_name==NULL || hoststatus_hashlist==NULL)
return NULL;
- for(temp_hoststatus=hoststatus_hashlist[hashfunc(host_name,NULL,HOSTSTATUS_HASHSLOTS)];temp_hoststatus && compare_hashdata(temp_hoststatus->host_name,NULL,host_name,NULL)<0;temp_hoststatus=temp_hoststatus->nexthash);
+ for(temp_hoststatus=hoststatus_hashlist[hashfunc(host_name,NULL,HOSTSTATUS_HASHSLOTS)];temp_hoststatus && compare_strings(temp_hoststatus->host_name,host_name)<0;temp_hoststatus=temp_hoststatus->nexthash);
- if(temp_hoststatus && (compare_hashdata(temp_hoststatus->host_name,NULL,host_name,NULL)==0))
+ if(temp_hoststatus && (compare_strings(temp_hoststatus->host_name,host_name)==0))
return temp_hoststatus;
return NULL;
--
1.6.4
------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
trial. Simplify your report design, integration and deployment - and focus on
what you do best, core application coding. Discover what's new with
Crystal Reports now. http://p.sf.net/sfu/bobj-july
More information about the Developers
mailing list