On Thu, Feb 04, 2010 at 10:52:43PM -0500, Rafi Rubin wrote:
+
> + if (matched < nd->contact_count) {
> + for (i = 0; i < nd->contact_count; i++) {
> + if (nd->contacts[i].logical_id < 0) {
> + for (j = 0; j < nd->prev_contact_count;
> + j++) {
> + if (nd->
> + prev_contacts[j].confidence
> + &&
> + (contact_map
> + [nd->
> + prev_contacts
> + [j].logical_id] < 0)
> + &&
> + (abs
> + (nd->contacts[i].x -
> + nd->prev_contacts[j].x) <
> + nd->max_width)
> + &&
> + (abs
> + (nd->contacts[i].y -
> + nd->prev_contacts[j].y) <
> + nd->max_height)) {
> + nd->contacts
> + [i].logical_id =
> + nd->prev_contacts
> + [j].logical_id;
> + contact_map
> + [nd->prev_contacts
> + [j].logical_id]
> + = i;
> + matched++;
OK, this kind code just makes me want to poke my eyes with a fork... 5
times...
Seriously, either factor it out into a nice function, or say "screw it"
to the 80 columt limit, or maybe both. Anything but this.
Thanks.
--
Dmitry
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
