Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#16 Indicate on diffs if a line is longer than 80 characters

open
nobody
5
2004-06-23
2004-06-23
Nathanial Drake
No

A cool feature would be to have the diffs somehow show
if a line goes over 80 characters (it could be
configurable). One of the things we look for in
reviews is compliance with coding standards (one of
which is an 80 column limit).

Thanks!

Discussion

  • David Sitsky
    David Sitsky
    2006-05-24

    Logged In: YES
    user_id=208928

    In response to a request from tennis AT cisco DOT com.

    Hi there,

    This has been asked before - I'll add this email to the
    existing request item.

    http://sourceforge.net/tracker/index.php?func=detail&aid=978531&group_id=41136&atid=42986

    In theory, we should be able to create a new topic listener
    which is fired on topic creation, which perhaps could add
    comments automatically to new lines which are greater than
    the specified parameter, if it exists.

    It may be a while before I look at this though - there are a
    number of other things on my plate right now.

    Cheers,
    David

    Tennis Smith (tennis) wrote:
    > Hi,
    >
    >
    >
    > I really like CodeStriker, it’s the best review tool I’ve
    ever seen.
    >
    >
    >
    > In my environment, we have a coding standard which
    requires that no line
    > be longer than 80 columns. Can CS flag that in a code
    review? Or, can
    > CS simply put a line at column 80 to tell the user when
    code exceeds it?
    >
    >
    >
    > TIA,
    >
    > -Tennis
    >