Primary/Secondary Link lists always started

The Primary/Secondary Link lists are started whether or not there are links to be displayed. This creates a validation error since unordered lists must contain items.

 26: <div id="top-nav">
 27: <ul id="primary">
 28: </ul>
 29:
 30: <ul id="secondary">
 31: </ul>
 32: </div>

In /themes/friendselectric/page.tpl.php

<div id="top-nav">
    <?php if (is_array($primary_links)): ?>
      <ul id="primary">
      <?php foreach ($primary_links as $link): ?>
        <li><?php print phptemplate_wrap_links($link, 2); ?></li>
      <?php endforeach; ?>
      </ul>
    <?php endif; ?>
    <?php if (is_array($secondary_links)): ?>
      <ul id="secondary">
      <?php foreach (array_reverse($secondary_links) as $link): ?>
        <li><?php print phptemplate_wrap_links($link, 2); ?></li>
      <?php endforeach; ?>
      </ul>
    <?php endif; ?>
  </div>

Is patched to --->

  <div id="top-nav">
    <?php if (is_array($primary_links) && !empty($primary_links)): ?>
      <ul id="primary">
      <?php foreach ($primary_links as $link): ?>
        <li><?php print phptemplate_wrap_links($link, 2); ?></li>
      <?php endforeach; ?>
      </ul>
    <?php endif; ?>
    <?php if (is_array($secondary_links) && !empty($secondary_links)): ?>
      <ul id="secondary">
      <?php foreach (array_reverse($secondary_links) as $link): ?>
        <li><?php print phptemplate_wrap_links($link, 2); ?></li>
      <?php endforeach; ?>
      </ul>
    <?php endif; ?>
  </div>

Patch is provided.

Let me know if something is wrong, since this is my first patch.

-Alan

P.S. Is there any reason the entire "top-nav" div shouldn't be displayed when there are no links? I didn't want to do this since I wasn't sure if it would break anything.