Re: [RESULTS] Code Style Guidelines

2006-11-17 Thread Adam Lally
Are there Eclipse configuration options necessary to meet any of these winners? One step ahead of you. See our podling web site. :) -Adam

Re: [RESULTS] Code Style Guidelines

2006-11-17 Thread eddie_epstein
> And the winners are... > > Braces: { > Looks like I'm outvoted on this one. > } > > Indentation: 2 spaces. No tabs! > > Member Variables: before methods > > Line Length: 100 (Preferred indent after wrapping: 8 spces) > > Imports: do not use package imports (e.g. import org.uima.*). order >

Code Conventions are up on our podling website

2006-11-17 Thread Adam Lally
Please review. Now if only our code conformed to our conventions. :) Also, I liked Thilo's idea of saying something about compiler warning settings and/or FindBugs rules. I'll let Thilo drive the proposal on that if he wants to. -Adam

[jira] Created: (UIMA-48) Reformat all source code to match adopted conventions

2006-11-17 Thread Adam Lally (JIRA)
Reformat all source code to match adopted conventions - Key: UIMA-48 URL: http://issues.apache.org/jira/browse/UIMA-48 Project: UIMA Issue Type: Task Reporter: Adam Lally We s

[jira] Resolved: (UIMA-47) Post code style guidelines on website

2006-11-17 Thread Adam Lally (JIRA)
[ http://issues.apache.org/jira/browse/UIMA-47?page=all ] Adam Lally resolved UIMA-47. Resolution: Fixed Website changes committed and extracted to people.apache.org. > Post code style guidelines on website > - > >

[jira] Updated: (UIMA-20) PearMerger unit test failure

2006-11-17 Thread Lev Kozakov (JIRA)
[ http://issues.apache.org/jira/browse/UIMA-20?page=all ] Lev Kozakov updated UIMA-20: Attachment: uima.example.DateTime.pear uima.example.RoomNumber.pear Both PEARs were recompiled with 1.4 compliance level > PearMerger unit test failure >

[jira] Created: (UIMA-47) Post code style guidelines on website

2006-11-17 Thread Adam Lally (JIRA)
Post code style guidelines on website - Key: UIMA-47 URL: http://issues.apache.org/jira/browse/UIMA-47 Project: UIMA Issue Type: Task Components: Website Reporter: Adam Lally

[RESULTS] Code Style Guidelines

2006-11-17 Thread Adam Lally
And the winners are... Braces: { Looks like I'm outvoted on this one. } Indentation: 2 spaces. No tabs! Member Variables: before methods Line Length: 100 (Preferred indent after wrapping: 8 spces) Imports: do not use package imports (e.g. import org.uima.*). order imports alphabetically

Re: anyone mind if I set up a confluence wiki for this project using the Apache Confluence wiki site?

2006-11-17 Thread Michael Baessler
I have no preferences for one of the two possibilities, but it sound good to me what Thilo wrote below about the permissions. -- Michael Thilo Goetz wrote: I'm lukewarm on wikis in general, but I do like the idea a little better of having one with the possibility of granting permissions for c

Re: Code Style Guidelines

2006-11-17 Thread Michael Baessler
[EMAIL PROTECTED] wrote: Braces: Each brace on a line by itself - Lally Braces: Opening brace not on a separate line, closing one on separate line - Schor Braces: Opening brace not on a separate line, closing one on separate line - Thilo (no strong feelings though) Braces: {

[jira] Closed: (UIMA-46) Duplicate feature name on supertype and subtype does not work if subtype definition comes first in descriptor

2006-11-17 Thread Adam Lally (JIRA)
[ http://issues.apache.org/jira/browse/UIMA-46?page=all ] Adam Lally closed UIMA-46. -- Resolution: Fixed Fixed by changing CasCreationUtils to always add features to supertypes before adding features to subtypes. > Duplicate feature name on supertype and subty

Re: [jira] Created: (UIMA-46) Duplicate feature name on supertype and subtype does not work if subtype definition comes first in descriptor

2006-11-17 Thread Adam Lally
On 11/17/06, Thilo Goetz <[EMAIL PROTECTED]> wrote: Is this something you would like to see fixed in the TypeSystemMgr, or in the code that calls the TypeSystemMgr? I believe I can probably fix it in the TypeSystemMgr without too much work. I already committed the fix for it in the code that

Re: [jira] Created: (UIMA-46) Duplicate feature name on supertype and subtype does not work if subtype definition comes first in descriptor

2006-11-17 Thread Thilo Goetz
Is this something you would like to see fixed in the TypeSystemMgr, or in the code that calls the TypeSystemMgr? I believe I can probably fix it in the TypeSystemMgr without too much work. --Thilo Adam Lally (JIRA) wrote: Duplicate feature name on supertype and subtype does not work if subty

[jira] Created: (UIMA-46) Duplicate feature name on supertype and subtype does not work if subtype definition comes first in descriptor

2006-11-17 Thread Adam Lally (JIRA)
Duplicate feature name on supertype and subtype does not work if subtype definition comes first in descriptor - Key: UIMA-46 URL: http://issues.apache.org/

Re: Code Style Guidelines

2006-11-17 Thread eddie_epstein
>> Braces: Each brace on a line by itself - Lally >> Braces: Opening brace not on a separate line, closing one on separate >> line - Schor >Braces: Opening brace not on a separate line, closing one on > separate line - Thilo (no strong feelings though) Braces: { same as schor & thilo (edd

Re: anyone mind if I set up a confluence wiki for this project using the Apache Confluence wiki site?

2006-11-17 Thread Thilo Goetz
I'm lukewarm on wikis in general, but I do like the idea a little better of having one with the possibility of granting permissions for certain areas. So +1 for confluence. --Thilo Adam Lally wrote: On 11/17/06, Marshall Schor <[EMAIL PROTECTED]> wrote: There are 2 wikis that can be used for

Re: anyone mind if I set up a confluence wiki for this project using the Apache Confluence wiki site?

2006-11-17 Thread Adam Lally
On 11/17/06, Marshall Schor <[EMAIL PROTECTED]> wrote: There are 2 wikis that can be used for projects: moin and confluence. I think I slightly prefer the confluence - I like the potential it has for collaborative authoring - I can see that being used for having the community help improve the do

anyone mind if I set up a confluence wiki for this project using the Apache Confluence wiki site?

2006-11-17 Thread Marshall Schor
There are 2 wikis that can be used for projects: moin and confluence. I think I slightly prefer the confluence - I like the potential it has for collaborative authoring - I can see that being used for having the community help improve the documentation. If no one objects, I'll ask the apache

Re: [jira] Updated: (UIMA-43) The ';' path separator char is not replaced with OS dependent char in installed PEAR

2006-11-17 Thread Thilo Goetz
Don't get me wrong, I would love to have a simpler process. However, while we're not sure where to draw the line, let's err on the side of caution. I hope that an ICLA is not too much of an impediment for people who want to contribute to UIMA. If we can figure out how to work that JIRA optio

Re: [jira] Updated: (UIMA-43) The ';' path separator char is not replaced with OS dependent char in installed PEAR

2006-11-17 Thread Marshall Schor
Re: Requiring ICLAs - there's a trade off here. If trivial things require contributors to file ICLAs, then we might put off getting contributions. There was some discussion on [EMAIL PROTECTED] also about this - something about Jira processing having a checkbox to say you have rights to contr

Re: [jira] Created: (UIMA-25) Implement better support for arrays of Feature Structures

2006-11-17 Thread Thilo Goetz
Marshall, do you recall why we put that code there? I only have a vague recollection that something was broken without this weird condition. In general, how are we going to proceed with the array stuff? Can/should we drop support for the old array names and simply move to the new scheme? Is

[ACTION REQUESTED] Michael and Eddie please respond to Coding Style Guidelines thread.

2006-11-17 Thread Adam Lally
Sorry to shout... but I'd like to get some closure on this so it would be good if all the committers gave their style preferences. Thanks! -Adam

Re: [jira] Updated: (UIMA-43) The ';' path separator char is not replaced with OS dependent char in installed PEAR

2006-11-17 Thread Thilo Goetz
For anything that's non-trivial (i.e., more than two lines) or non-obvious, even if less than two lines, we require an ICLA. The ICLA should be on file before the patch gets submitted. Strictly speaking, we need to ask people to resubmit their patches after their ICLA is on file. --Thilo A

[jira] Created: (UIMA-45) Review and clean up unit tests

2006-11-17 Thread Adam Lally (JIRA)
Review and clean up unit tests -- Key: UIMA-45 URL: http://issues.apache.org/jira/browse/UIMA-45 Project: UIMA Issue Type: Task Components: Build, Packaging and Test Reporter: Adam Lally

Re: Remaining occurrences of IBM

2006-11-17 Thread Adam Lally
On 11/17/06, Michael Baessler <[EMAIL PROTECTED]> wrote: so I would recommend that we just rename the IBMResultPrinter class to ResultPrinter. I will take care of that if all agree. Fine with me. While you're in there you might also want to take a look at what I did to TestPropertyReader in

[jira] Resolved: (UIMA-41) LowLevelCAS.ll_getTypeClass() needs to be updated for the new 2.0 types

2006-11-17 Thread Thilo Goetz (JIRA)
[ http://issues.apache.org/jira/browse/UIMA-41?page=all ] Thilo Goetz resolved UIMA-41. - Resolution: Invalid It's a non-issue, everything is implemented as it should. > LowLevelCAS.ll_getTypeClass() needs to be updated for the new 2.0 types > --

Re: Somebody please figure out how to do this better (was Re: svn commit: r475808 - in /incubator/uima/uimaj/trunk/uimaj-tools/src: main/java/org/apache/uima/tools/viewer/CasAnnotationViewer.java test

2006-11-17 Thread Thilo Goetz
I apologize, maybe I should get glasses. Everything's there, I just didn't see it. Marshall Schor wrote: Best not to ascribe blame for these, I think. This may have in fact been my oversight :-) -Marshall Thilo Goetz wrote: Adam Lally wrote: I fixed a bug (reported on the forum) that the

[jira] Closed: (UIMA-11) org.apache.itu package should be renamed

2006-11-17 Thread Thilo Goetz (JIRA)
[ http://issues.apache.org/jira/browse/UIMA-11?page=all ] Thilo Goetz closed UIMA-11. --- Resolution: Fixed Replaced occurrences of org.apache.itu with org.apache.uima.uimacpp, and occurrences of Taf with Uimacpp. > org.apache.itu package should be renamed > -

Re: Jira for minor issues?

2006-11-17 Thread Michael Baessler
I'm also think it is better to have a JIRA issue for all changes. -- Michael Marshall Schor wrote: I agree. -Marshall Thilo Goetz wrote: Adam Lally wrote: On 11/15/06, Thilo Goetz <[EMAIL PROTECTED]> wrote: I updated our website to fix a broken link without a Jira issue (didn't really thin

[jira] Commented: (UIMA-44) rename IBMResultPrinter class

2006-11-17 Thread Michael Baessler (JIRA)
[ http://issues.apache.org/jira/browse/UIMA-44?page=comments#action_12450737 ] Michael Baessler commented on UIMA-44: -- ResultPrinter is already used by JUnit itself, so I used UIMAResultPrinter. > rename IBMResultPrinter class > ---

[jira] Created: (UIMA-44) rename IBMResultPrinter class

2006-11-17 Thread Michael Baessler (JIRA)
rename IBMResultPrinter class - Key: UIMA-44 URL: http://issues.apache.org/jira/browse/UIMA-44 Project: UIMA Issue Type: Bug Components: Tools Reporter: Michael Baessler Assigned To:

Re: Remaining occurrences of IBM

2006-11-17 Thread Michael Baessler
Adam Lally wrote: I've replaced most occurrences of "IBM" now. Here's a list of what I didn't fix: 6) uimaj-test-util contains class IBMResultPrinter developed by Michael. Does this need to be here? Currently I don't see that we need the uimaj-test-util project when using Maven for our comm