Misha Koshelev <[email protected]> writes: > + for (i = 0; i < number_of_vertices; i++) > + { > + ok(compare_vec3(vertices[i].position, > mesh->vertices[i].position), "Test %s, vertex position %d, result (%g, %g, > %g), expected (%g, %g, %g)\n", name, i, vertices[i].position.x, > vertices[i].position.y, vertices[i].position.z, mesh->vertices[i].position.x, > mesh->vertices[i].position.y, mesh->vertices[i].position.z); > + ok(compare_vec3(vertices[i].normal, > mesh->vertices[i].normal), "Test %s, vertex normal %d, result (%g, %g, %g), > expected (%g, %g, %g)\n", name, i, vertices[i].normal.x, > vertices[i].normal.y, vertices[i].normal.z, mesh->vertices[i].normal.x, > mesh->vertices[i].normal.y, mesh->vertices[i].normal.z);
Please add some line breaks occasionally. 100 chars is a reasonable limit, a bit more is tolerated, but 300 is too much. > + /* free lookup tables */ > + free_sincos_table(&phi); > + free_sincos_table(&theta); > + > + /* success */ > + return TRUE; There's no need for that sort of comments, the purpose of that code is obvious. -- Alexandre Julliard [email protected]
