X-Git-Url: https://git.m6w6.name/?p=m6w6%2Fext-psi;a=blobdiff_plain;f=src%2Ftypes%2Fdecl_struct.c;h=35818b2d557167d2a494ca7cfa8333dbacac393f;hp=d5309b9cb041d95337bfe3cbd3db0286bed78f5b;hb=9aeae2c241e303c537d8be18dd6c9f8776b2f044;hpb=c9384515a81cb64d345b299908b2852f51bb8e6e

diff --git a/src/types/decl_struct.c b/src/types/decl_struct.c
index d5309b9..35818b2 100644
--- a/src/types/decl_struct.c
+++ b/src/types/decl_struct.c
@@ -81,7 +81,7 @@ struct psi_decl_arg *psi_decl_struct_get_arg(struct psi_decl_struct *s,
 bool psi_decl_struct_validate(struct psi_data *data, struct psi_decl_struct *s,
 		struct psi_validate_scope *scope)
 {
-	size_t i, pos, len, size, align;
+	size_t i, pos = 0, len = 0;
 	struct psi_decl_arg *darg, *prev_arg;
 
 	if (!s) {
@@ -92,12 +92,19 @@ bool psi_decl_struct_validate(struct psi_data *data, struct psi_decl_struct *s,
 	}
 
 	if (!s->size && !psi_plist_count(s->args)) {
+		/* TODO: return true and check those structs are only used by address */
+		/* suppress needless warning
+		data->error(data, s->token, PSI_WARNING, "Empty struct %s",
+				s->name);
+				*/
 		return false;
 	}
 
 	psi_validate_scope_add_struct(scope, s->name, s);
 
 	for (i = 0; psi_plist_get(s->args, i, &darg); ++i) {
+		size_t align;
+
 		darg->var->arg = darg;
 
 		if (!psi_decl_arg_validate(data, darg, scope)) {
@@ -189,6 +196,8 @@ bool psi_decl_struct_validate(struct psi_data *data, struct psi_decl_struct *s,
 	}
 
 	if (psi_plist_count(s->args)) {
+		size_t size;
+
 		psi_plist_sort(s->args, psi_layout_sort_cmp, NULL);
 		psi_plist_get(s->args, psi_plist_count(s->args) - 1, &darg);