Improve FPP rendering process

Hello

Currently, the FPP renders it's $fields content in the preprocess function template_preprocess_fieldable_panels_pane() with this code:

  $vars['fields'] = '';
  foreach ($vars['content'] as $content) {
    $vars['fields'] .= render($content);
  }

and and then prints in the template fieldable-panels-pane.tpl.php using this line: <?php print $fields; ?>

I'm currently altering the markup in my own preprocess hook_preprocess_fieldable_panels_pane(), but have to also worry about *rendering that markup* again in my preprocess.

function mymodule_preprocess_fieldable_panels_pane(&$vars) {
  if ($my_conditions) {
    $vars['content']['my_field'][0]['#markup'] = l($vars['content']['my_field'][0]['#markup'], 'http://drupal.org');
    $vars['fields'] = '';
    foreach ($vars['content'] as $content) {
      $vars['fields'] .= render($content);
    }
  }
}

This is not the standard way to render an entity and it could hit the site performance because FPP content is rendered twice, (once in template_preprocess_fieldable_panels_pane() and then again in mymodule_preprocess_fieldable_panels_pane()).

I propose we remove the $fields string and render the $content variable in the template file, like other entity types do.

What do you think?

Regards