% chance you'll hack shindig on a production server * % chance it
doesnt have an ssh shell * % chance it doesnt even have X + XTerm =
not worth wrapping on 80 characters :)
On Jun 10, 2008, at 10:31 PM, Gonzalo Aune wrote:
Its not about a monitor, imagine you have to take a look at some
code on a
production system in a unix system with only its terminal, the 120
lines
could fit in there ;)
Regards.
On Tue, Jun 10, 2008 at 4:45 PM, Brian McCallister
<[EMAIL PROTECTED]> wrote:
I use 120, personally. If anyone is still stuck on an 80 column
terminal,
email me offline and I'll send you a monitor :-)
-Brian
On Tue, Jun 10, 2008 at 10:14 AM, Kevin Brown <[EMAIL PROTECTED]>
wrote:
On Tue, Jun 10, 2008 at 8:59 AM, Ropu <[EMAIL PROTECTED]> wrote:
and for PHP stuff?
Each language is different. PHP is far less verbose than Java, so
it's
not
as big of an issue. Java has this ugly mess littered all over the
place:
for (Map.Entry<String, Collection<String>> entry :
request.getHeaders().entrySet() {
}
Whereas in PHP similar code is:
foreach ($request->getHeaders() as $key=>$values) {
}
This thread is only dealing with the Java case in order to get
things
consistent going forward.
i would recommend 100perLine too.
but many code must be rewriten.
ropu
On Tue, Jun 10, 2008 at 4:37 AM, Ian Boston <[EMAIL PROTECTED]> wrote:
2008/6/10 Kevin Brown <[EMAIL PROTECTED]>:
On Tue, Jun 10, 2008 at 1:52 AM, Ian Boston <[EMAIL PROTECTED]>
wrote:
+1 for 100, (but not a committer, so not a strongly held view),
however
+1 for consistency to reduce ease of patching.
Reduce ease of patching? :)
Oops, increase :)
Sorry
Need caffeine.
Ian
2008/6/10 Paul Lindner <[EMAIL PROTECTED]>:
I'm in favor of 100. I find that even with 2 char indents that
we
line
wrap
more code than I'm used to. That causes readability to suffer.
On Jun 10, 2008, at 1:22 AM, Kevin Brown wrote:
Our style guide calls for 80 columns (
http://cwiki.apache.org/SHINDIGxSITE/java-style.html#JavaStyle-Linewrapping
.)
Unfortunately, this hasn't been followed very well, so that
leaves
us
with
two options:
- Modify all the existing code to 80 columns.
- Up the limit to 100 columns (covers existing code, still
narrow
enough
for
most terminals).
Any strong opinions on this one?
Paul Lindner
[EMAIL PROTECTED]
--
.-. --- .--. ..-
R o p u
