Commerce File links to a non-existent file/%file/download when used with File Entity

The code in commerce_file_menu() reads as follows:

/**
 * Implements hook_menu_alter().
 *
 * Provides a file download callback in case file_entity is missing.
 */
function commerce_file_menu_alter(&$items) {
  // Added via hook_menu_alter() in order to override the same path
  // when provided by file_download_count (since that version doesn't force
  // the file to be downloaded).
  if (!module_exists('file_entity')) {
    $items['file/%file/download'] = array(
      // ...
    );
  }
}

The problem is that the File Entity module doesn't actually define a file/%file/download URL, but Commerce File still provides a display formatter for file entities that links to this URL. This means it falls back to trying to show the file/%file page instead, which should be a 403 in most cases but has resulted in what I can only assume is a redirect loop on a site I'm working on manifested as a PHP memory limit error.

I'm thinking the solution isn't to link to file/%file/download when File Entity exists and instead to just link directly to the file URL. I fixed the site I was working on by changing the user Files tab to use the "Generic files" display formatter instead of "Commerce file". However, that doesn't work for the checkout pane, which is apparently not derived from Views at all, so I'm just going to disable the "License completion message" checkout pane and update the checkout completion message to point to the user account page instead.