Hans Leidekker (@hans) commented about dlls/wbemprox/builtin.c:
- enum fill_status status = FILL_STATUS_UNFILTERED;
- UINT row = 0;
- UINT64 max_capacity;
- if (!resize_table( table, 1, sizeof(*rec) )) return FILL_STATUS_FAILED;
- max_capacity = get_total_physical_memory();
- rec = (struct record_physicalmemoryarray *)table->data;
- rec->caption = L"Physical Memory Array";
- rec->max_capacity = max_capacity > 0x400000 ? 0x400000 : max_capacity;
- rec->max_capacity_ex = max_capacity;
- rec->model = NULL;
- rec->name = L"Physical Memory Array";
- rec->status = NULL;
- if (!match_row( table, row, cond, &status )) free_row_values( table, row );
There's no need to zero-initialize fields. Otherwise this looks good.