<div id="xxx" is output twice in GMap theming - breaks html validator

The GMap module generates code that breaks the html validator. Not a major problem but still not clever.

The ID of the div that wraps around the map gets the id attribute twice. It seems the theme_gmap function passes the id in the element[#attributes] array to drupal_attributes() as well using the element[#id] to assign the divs id.

Lines 1103 to 1108 in gmap.module…

 $element['#attributes']['class'] = trim($element['#attributes']['class'] .'gmap gmap-map gmap-'. $map['id'] .'-gmap');

  // Some markup parsers (IE) don't handle empty inners well. Use the space to let users know javascript is required.
  // @@@ Bevan sez: Google static maps could be useful here.
  // @@@ Bdragon sez: Yeah, would be nice, but hard to guarantee functionality. Not everyone uses the static markerloader.
  $o = '<div id="'. $element['#id'] .'" style="'. implode('; ', $style) .';"'. drupal_attributes($element['#attributes']) .'>'. t('Javascript is required to view this map.') .'&lt;/div&gt;';

Not sure if there is a reason for this duplication (Javascript isn't my strong point), but simply removing the id="'. $element['#id'] .' from line 1108 should fix it.
I can't test at the moment because I'm using a macro in a body field so the GMap div ends up inside a paragraph tag, (& also fails validation).

Drew