aboutsummaryrefslogtreecommitdiffstats
path: root/net/dccp/feat.c
diff options
context:
space:
mode:
authorRusty Russell <rusty@rustcorp.com.au>2011-12-19 08:56:45 -0500
committerDavid S. Miller <davem@davemloft.net>2011-12-19 22:27:29 -0500
commit3db1cd5c05f35fb43eb134df6f321de4e63141f2 (patch)
tree960039f3f4f0a524b37e94434624da154859bc64 /net/dccp/feat.c
parenta8e510f682fe6d7671c11887e07c55f86caaf3c1 (diff)
net: fix assignment of 0/1 to bool variables.
DaveM said: Please, this kind of stuff rots forever and not using bool properly drives me crazy. Joe Perches <joe@perches.com> gave me the spatch script: @@ bool b; @@ -b = 0 +b = false @@ bool b; @@ -b = 1 +b = true I merely installed coccinelle, read the documentation and took credit. Signed-off-by: Rusty Russell <rusty@rustcorp.com.au> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dccp/feat.c')
-rw-r--r--net/dccp/feat.c16
1 files changed, 8 insertions, 8 deletions
diff --git a/net/dccp/feat.c b/net/dccp/feat.c
index 23cea0ee3101..78a2ad70e1b0 100644
--- a/net/dccp/feat.c
+++ b/net/dccp/feat.c
@@ -490,8 +490,8 @@ static int dccp_feat_push_change(struct list_head *fn_list, u8 feat, u8 local,
490 new->feat_num = feat; 490 new->feat_num = feat;
491 new->is_local = local; 491 new->is_local = local;
492 new->state = FEAT_INITIALISING; 492 new->state = FEAT_INITIALISING;
493 new->needs_confirm = 0; 493 new->needs_confirm = false;
494 new->empty_confirm = 0; 494 new->empty_confirm = false;
495 new->val = *fval; 495 new->val = *fval;
496 new->needs_mandatory = mandatory; 496 new->needs_mandatory = mandatory;
497 497
@@ -517,12 +517,12 @@ static int dccp_feat_push_confirm(struct list_head *fn_list, u8 feat, u8 local,
517 new->feat_num = feat; 517 new->feat_num = feat;
518 new->is_local = local; 518 new->is_local = local;
519 new->state = FEAT_STABLE; /* transition in 6.6.2 */ 519 new->state = FEAT_STABLE; /* transition in 6.6.2 */
520 new->needs_confirm = 1; 520 new->needs_confirm = true;
521 new->empty_confirm = (fval == NULL); 521 new->empty_confirm = (fval == NULL);
522 new->val.nn = 0; /* zeroes the whole structure */ 522 new->val.nn = 0; /* zeroes the whole structure */
523 if (!new->empty_confirm) 523 if (!new->empty_confirm)
524 new->val = *fval; 524 new->val = *fval;
525 new->needs_mandatory = 0; 525 new->needs_mandatory = false;
526 526
527 return 0; 527 return 0;
528} 528}
@@ -1155,7 +1155,7 @@ static u8 dccp_feat_change_recv(struct list_head *fn, u8 is_mandatory, u8 opt,
1155 } 1155 }
1156 1156
1157 if (dccp_feat_reconcile(&entry->val, val, len, server, true)) { 1157 if (dccp_feat_reconcile(&entry->val, val, len, server, true)) {
1158 entry->empty_confirm = 0; 1158 entry->empty_confirm = false;
1159 } else if (is_mandatory) { 1159 } else if (is_mandatory) {
1160 return DCCP_RESET_CODE_MANDATORY_ERROR; 1160 return DCCP_RESET_CODE_MANDATORY_ERROR;
1161 } else if (entry->state == FEAT_INITIALISING) { 1161 } else if (entry->state == FEAT_INITIALISING) {
@@ -1171,10 +1171,10 @@ static u8 dccp_feat_change_recv(struct list_head *fn, u8 is_mandatory, u8 opt,
1171 defval = dccp_feat_default_value(feat); 1171 defval = dccp_feat_default_value(feat);
1172 if (!dccp_feat_reconcile(&entry->val, &defval, 1, server, true)) 1172 if (!dccp_feat_reconcile(&entry->val, &defval, 1, server, true))
1173 return DCCP_RESET_CODE_OPTION_ERROR; 1173 return DCCP_RESET_CODE_OPTION_ERROR;
1174 entry->empty_confirm = 1; 1174 entry->empty_confirm = true;
1175 } 1175 }
1176 entry->needs_confirm = 1; 1176 entry->needs_confirm = true;
1177 entry->needs_mandatory = 0; 1177 entry->needs_mandatory = false;
1178 entry->state = FEAT_STABLE; 1178 entry->state = FEAT_STABLE;
1179 return 0; 1179 return 0;
1180 1180