This Bugzilla instance is a read-only archive of historic NetBeans bug reports. To report a bug in NetBeans please follow the project's instructions for reporting issues.

Bug 8742 - Method parameters should be separated by ', ' (comma + space)
Summary: Method parameters should be separated by ', ' (comma + space)
Status: CLOSED FIXED
Alias: None
Product: java
Classification: Unclassified
Component: Unsupported (show other bugs)
Version: 3.x
Hardware: Other Windows ME/2000
: P4 enhancement (vote)
Assignee: issues@java
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2000-12-04 10:21 UTC by Jan Becicka
Modified: 2007-09-26 09:14 UTC (History)
0 users

See Also:
Issue Type: ENHANCEMENT
Exception Reporter:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Becicka 2000-12-04 10:21:25 UTC
[release 3.1-20] [1160]
I suggest to separate method parameters by ', ' (comma + space), not only by ','

E.g. foo(int a, int b) instead of foo(int a,int b)
Comment 1 Tomas Hurka 2001-01-12 13:23:59 UTC
I suggest to implement this enhacement by patch bellow and implement ", "
behaviour in ConstructorElement, MethodElement and MethodChildrens classes.

Index: java/src/org/netbeans/modules/java/wizard/MethodsChildren.java
===================================================================
RCS file: /cvs/java/src/org/netbeans/modules/java/wizard/MethodsChildren.java,v
retrieving revision 1.1
diff -r1.1 MethodsChildren.java
22c22
<     private static ElementFormat METHOD_FORMAT = new
ElementFormat("{n}({p,,,\",\"})");
---
>     private static ElementFormat METHOD_FORMAT = new
ElementFormat("{n}({p})");
Index: openide/src/org/openide/src/ConstructorElement.java
===================================================================
RCS file: /cvs/openide/src/org/openide/src/ConstructorElement.java,v
retrieving revision 1.30
diff -r1.30 ConstructorElement.java
27c27
<         new ElementFormat("{m,,\" \"}{n}({a,,,\",\"}){e,\" throws \",}"); //
NOI18N
---
>         new ElementFormat("{m,,\" \"}{n}({a}){e,\" throws \",}"); // NOI18N
Index: openide/src/org/openide/src/MethodElement.java
===================================================================
RCS file: /cvs/openide/src/org/openide/src/MethodElement.java,v
retrieving revision 1.28
diff -r1.28 MethodElement.java
29c29
<         new ElementFormat("{m,,\" \"}{r} {n}({a,,,\",\"}){e,\" throws \",}");
// NOI18N
---
>         new ElementFormat("{m,,\" \"}{r} {n}({a}){e,\" throws \",}"); //
NOI18N
Comment 2 Jan Becicka 2001-01-31 14:16:59 UTC
[010130_1] Verified
Comment 3 Quality Engineering 2003-07-01 13:15:33 UTC
Resolved for 3.4.x or earlier, no new info since then -> closing.