degesch: further refactor MODE processing
This commit is contained in:
		
							parent
							
								
									2b258007f0
								
							
						
					
					
						commit
						de2eff7399
					
				
							
								
								
									
										92
									
								
								degesch.c
									
									
									
									
									
								
							
							
						
						
									
										92
									
								
								degesch.c
									
									
									
									
									
								
							| @ -3970,7 +3970,7 @@ struct mode_processor | ||||
| }; | ||||
| 
 | ||||
| /// Process a single mode character
 | ||||
| typedef bool (*mode_processor_fn) (struct mode_processor *, char); | ||||
| typedef bool (*mode_processor_apply_fn) (struct mode_processor *); | ||||
| 
 | ||||
| static const char * | ||||
| mode_processor_next_param (struct mode_processor *self) | ||||
| @ -3982,7 +3982,7 @@ mode_processor_next_param (struct mode_processor *self) | ||||
| 
 | ||||
| static void | ||||
| mode_processor_run (struct mode_processor *self, | ||||
| 	char **params, mode_processor_fn apply_cb) | ||||
| 	char **params, mode_processor_apply_fn apply_cb) | ||||
| { | ||||
| 	self->params = params; | ||||
| 
 | ||||
| @ -3994,12 +3994,28 @@ mode_processor_run (struct mode_processor *self, | ||||
| 		{ | ||||
| 			if      (self->mode_char == '+') self->adding = true; | ||||
| 			else if (self->mode_char == '-') self->adding = false; | ||||
| 			else if (!apply_cb (self, self->mode_char)) | ||||
| 			else if (!apply_cb (self)) | ||||
| 				break; | ||||
| 		} | ||||
| 	} | ||||
| } | ||||
| 
 | ||||
| static void | ||||
| mode_processor_toggle (struct mode_processor *self, struct str *modes) | ||||
| { | ||||
| 	const char *pos = strchr (modes->str, self->mode_char); | ||||
| 	if (self->adding == !!pos) | ||||
| 		return; | ||||
| 
 | ||||
| 	if (self->adding) | ||||
| 	{ | ||||
| 		str_append_c (modes, self->mode_char); | ||||
| 		// TODO: sort the modes
 | ||||
| 	} | ||||
| 	else | ||||
| 		str_remove_slice (modes, pos - modes->str, 1); | ||||
| } | ||||
| 
 | ||||
| // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 | ||||
| 
 | ||||
| static void | ||||
| @ -4044,10 +4060,8 @@ mode_processor_do_param_always (struct mode_processor *self) | ||||
| 		return; | ||||
| 
 | ||||
| 	char key[2] = { self->mode_char, 0 }; | ||||
| 	if (self->adding) | ||||
| 		str_map_set (&self->channel->param_modes, key, xstrdup (param)); | ||||
| 	else | ||||
| 		str_map_set (&self->channel->param_modes, key, NULL); | ||||
| 	str_map_set (&self->channel->param_modes, key, | ||||
| 		self->adding ? xstrdup (param) : NULL); | ||||
| } | ||||
| 
 | ||||
| static void | ||||
| @ -4058,47 +4072,24 @@ mode_processor_do_param_when_set (struct mode_processor *self) | ||||
| 		return; | ||||
| 
 | ||||
| 	char key[2] = { self->mode_char, 0 }; | ||||
| 	if (self->adding) | ||||
| 		str_map_set (&self->channel->param_modes, key, xstrdup (param)); | ||||
| 	else | ||||
| 		str_map_set (&self->channel->param_modes, key, NULL); | ||||
| } | ||||
| 
 | ||||
| static void | ||||
| mode_processor_do_param_never (struct mode_processor *self) | ||||
| { | ||||
| 	struct str *modes = &self->channel->no_param_modes; | ||||
| 	const char *pos = strchr (modes->str, self->mode_char); | ||||
| 	if (self->adding == !!pos) | ||||
| 		return; | ||||
| 
 | ||||
| 	if (self->adding) | ||||
| 	{ | ||||
| 		str_append_c (modes, self->mode_char); | ||||
| 		// TODO: sort the modes
 | ||||
| 	} | ||||
| 	else | ||||
| 		str_remove_slice (modes, pos - modes->str, 1); | ||||
| 	str_map_set (&self->channel->param_modes, key, | ||||
| 		self->adding ? xstrdup (param) : NULL); | ||||
| } | ||||
| 
 | ||||
| static bool | ||||
| mode_processor_apply_channel (struct mode_processor *self, char mode_char) | ||||
| mode_processor_apply_channel (struct mode_processor *self) | ||||
| { | ||||
| 	struct server *s = self->s; | ||||
| 
 | ||||
| 	if      (strchr (s->irc_chanuser_modes,           mode_char)) | ||||
| 		mode_processor_do_user           (self); | ||||
| 
 | ||||
| 	else if (strchr (s->irc_chanmodes_list,           mode_char)) | ||||
| 		// Nothing to do here, really
 | ||||
| 	if      (strchr (self->s->irc_chanuser_modes,           self->mode_char)) | ||||
| 		mode_processor_do_user (self); | ||||
| 	else if (strchr (self->s->irc_chanmodes_list,           self->mode_char)) | ||||
| 		// Nothing to do here, just skip the next argument if there's any
 | ||||
| 		(void) mode_processor_next_param (self); | ||||
| 
 | ||||
| 	else if (strchr (s->irc_chanmodes_param_always,   mode_char)) | ||||
| 		mode_processor_do_param_always   (self); | ||||
| 	else if (strchr (s->irc_chanmodes_param_when_set, mode_char)) | ||||
| 	else if (strchr (self->s->irc_chanmodes_param_always,   self->mode_char)) | ||||
| 		mode_processor_do_param_always (self); | ||||
| 	else if (strchr (self->s->irc_chanmodes_param_when_set, self->mode_char)) | ||||
| 		mode_processor_do_param_when_set (self); | ||||
| 	else if (strchr (s->irc_chanmodes_param_never,    mode_char)) | ||||
| 		mode_processor_do_param_never    (self); | ||||
| 	else if (strchr (self->s->irc_chanmodes_param_never,    self->mode_char)) | ||||
| 		mode_processor_toggle (self, &self->channel->no_param_modes); | ||||
| 	else | ||||
| 		// It's not safe to continue, results could be undesired
 | ||||
| 		return false; | ||||
| @ -4116,22 +4107,9 @@ irc_handle_mode_channel | ||||
| // - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
 | ||||
| 
 | ||||
| static bool | ||||
| mode_processor_apply_user (struct mode_processor *self, char mode_char) | ||||
| mode_processor_apply_user (struct mode_processor *self) | ||||
| { | ||||
| 	struct server *s = self->s; | ||||
| 
 | ||||
| 	struct str *modes = &s->irc_user_mode; | ||||
| 	const char *pos = strchr (modes->str, mode_char); | ||||
| 	if (self->adding == !!pos) | ||||
| 		return true; | ||||
| 
 | ||||
| 	if (self->adding) | ||||
| 	{ | ||||
| 		str_append_c (modes, mode_char); | ||||
| 		// TODO: sort the modes
 | ||||
| 	} | ||||
| 	else | ||||
| 		str_remove_slice (modes, pos - modes->str, 1); | ||||
| 	mode_processor_toggle (self, &self->s->irc_user_mode); | ||||
| 	return true; | ||||
| } | ||||
| 
 | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user