couchdb-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jan Lehnardt (JIRA)" <>
Subject [jira] Commented: (COUCHDB-669) Minor code cleanup in couch_httpd_auth
Date Tue, 23 Feb 2010 18:42:27 GMT


Jan Lehnardt commented on COUCHDB-669:

Hi Joel,

this is a good patch, you're doing it right :)

Only a few comments: 

Line 39 of the patch adds a new empty line with one indenting level of spaces. It is customary
to keep separate patches for code changes and whitespace changes, since code changes are hard
to spot when you only change whitespace in a line. Now your whitespace is in an isolated place
so it is hardly a big deal, but it can make larger patches hard to read. No biggie :)

Secondly, we try to avoid trailing whitespace, so if you want to add a new empty line (which
is fine), try not having extra spaces in there.

In concise form, these are summed up in our coding guidelines:

Other than that the patch is looking good, thanks! :)

> Minor code cleanup in couch_httpd_auth
> --------------------------------------
>                 Key: COUCHDB-669
>                 URL:
>             Project: CouchDB
>          Issue Type: Improvement
>            Reporter: Joel Clark
>            Priority: Trivial
>         Attachments: COUCH-669.diff
>   Original Estimate: 0.08h
>  Remaining Estimate: 0.08h
> While experimenting with a custom handler I found some code repeated several times. 
Figured I'd get my feet wet with both Couch and Erlang at the same time and submit a patch
that removes the repetition.  Please review and apply as you see fit.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message