Review of the 7.x-1.x branch:

./theme-settings.php: all functions should be prefixed with your module/theme name to avoid name clashes. See http://drupal.org/node/318#naming

function sf_theme_settings_validate($form, &$form_state) {
function sf_theme_settings_submit($form, &$form_state) {
function sf_layout_styles($method, $sidebar_first, $sidebar_second) {

Run coder to check your style, some issues were found (please check the Drupal coding standards). See attachment.
Drupal Code Sniffer has found some code style issues (please check the Drupal coding standards). See attachment.

This automated report was generated with PAReview.sh, your friendly project application review script. 
You can also use the online version to check your project. Get a review bonus and we will come back to your application sooner.


Severity minor, Drupal Commenting Standards, Internationalization, Drupal Security Checks, Drupal SQL Standards, Drupal Coding Standards

sites/all/modules/pareview_temp/./test_candidate/theme-settings.php:
+392: [normal] The $string argument to t() should not begin or end with a space.

Status Messages:
Coder found 23 projects, 23 files, 1 normal warnings, 0 warnings were flagged to be ignored

FILE: ...-pareview/sites/all/modules/pareview_temp/test_candidate/css/screen.css
--------------------------------------------------------------------------------
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
794 | ERROR | No style definition found on line; check for missing colon
--------------------------------------------------------------------------------

FILE: ...sites/all/modules/pareview_temp/test_candidate/inc/template.process.inc
--------------------------------------------------------------------------------
FOUND 2 ERROR(S) AFFECTING 2 LINE(S)
--------------------------------------------------------------------------------
322 | ERROR | An operator statement must be followed by a single space
512 | ERROR | An operator statement must be followed by a single space
--------------------------------------------------------------------------------

FILE: ...s/all/modules/pareview_temp/test_candidate/sfstarter/SFSTARTER.info.txt
--------------------------------------------------------------------------------
FOUND 0 ERROR(S) AND 1 WARNING(S) AFFECTING 1 LINE(S)
--------------------------------------------------------------------------------
2 | WARNING | Line exceeds 80 characters; contains 116 characters
--------------------------------------------------------------------------------