[pacman-dev] [PATCH] pacsort: safer sizeof types when allocating memory

Julien Ramseier j.ramseier at gmail.com
Sat Jan 30 12:29:15 UTC 2016


> Le 30 janv. 2016 à 10:00, Allan McRae <allan at archlinux.org> a écrit :
> 
> Why is this safer?

Safer is maybe not the most appropriate word.
This simply prevents mistakes by having to write the correct type twice.

Using char* instead of void* is also semantically incorrect:
-	void **new_list = realloc(list->list, newsz * sizeof(char *));
+	void **new_list = realloc(list->list, newsz * sizeof(*new_list));


- Julien


More information about the pacman-dev mailing list