bugGNUstep - Bugs: bug #9955, Gorm parse outlets redundantly.

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #9955: Gorm parse outlets redundantly.

Submitted by:  -Deleted Account- <yjchen>
Submitted on:  Tue 10 Aug 2004 04:12:58 PM UTC  
 
Category: GormSeverity: 3 - Normal
Item Group: BugStatus: Fixed
Privacy: PublicAssigned to: Gregory John Casamento <gcasa>
Open/Closed: Closed

(Jump to the original submission Jump to the original submission)

Sat 27 Nov 2004 11:14:02 AM UTC, comment #6:

I have added a new class parser which corrects this issue. The class in the description now parses correctly.

I have also attached the file I created to test with.

GJC

Gregory John Casamento <gcasa>
Project AdministratorIn charge of this item.
Fri 26 Nov 2004 10:40:17 AM UTC, comment #5:

I am in the process of writing a new class parser. I'm wondering if the submitter could attach an example of the header which was causing the issue. GJC

Gregory John Casamento <gcasa>
Project AdministratorIn charge of this item.
Fri 13 Aug 2004 10:37:58 PM UTC, comment #4:

I've tried the attached parser. It has promise and I believe it's got the right idea.

I believe, however, a general objc header parsing framework should be written. Since I want to keep Gorm's external dependency list short, I am going to write the library as another lib which is built and installed with Gorm (like the GormLib/InterfaceBuilder) library.

GJC

Gregory John Casamento <gcasa>
Project AdministratorIn charge of this item.
Fri 13 Aug 2004 10:24:53 PM UTC, comment #3:

Several people have complained about the header parser causing issues. I have, therefore, disabled it in current Gorm CVS. I am currently in the process of re-writing the class parser. I should be done sometime this weekend.

In the mean time, people should add classes by hand. In case anyone is wondering the procedure is (and has always been):

1) create a new document (app, inspector, etc...)
2) Click on the classes toolbar item
3) Select the parent class
4) Click "Gorm->Classes->Subclass"
5) Rename the new class from NewClass to the name you want.
6) Use the class inspector or the outline view+classes menu to add actions/outlets to the new class.

Thanks, GJC

Gregory John Casamento <gcasa>
Project AdministratorIn charge of this item.
Wed 11 Aug 2004 04:08:52 PM UTC, comment #2:

I made a simple header parser (see attached file).
It uses a general code parser in CodeEditor and MyLibrary.
The idea is the same as SAX.
It only parses the outlet starting with "IBOutlet" or "id".
It handles both /* and // comment.
use 'opentool HeaderParser <header file>' to test.
It is just a demo and is not polished.
If you like this idea of parsing header,
we can make it better.

-Deleted Account- <yjchen>
Wed 11 Aug 2004 02:38:22 AM UTC, comment #1:

Thanks for the report. The class parser currently needs some revamping. GJC

Gregory John Casamento <gcasa>
Project AdministratorIn charge of this item.
Tue 10 Aug 2004 04:12:58 PM UTC, original submission:

1. Open gorm.
2. "Document" -> "New Module" -> "New Empty"
3. Select "Classes" in main window.
4. "Classes" -> "Load Class..." to load this header:

#include <AppKit/AppKit.h>

@interface LibraryLocationPanel: NSObject
{
IBOutlet id locationTextField;
IBOutlet id panel;
int buttonResult;
}

@end

5. Look at the outlet of this class in inspector.
The "panel" outlet appears redundantly.

-Deleted Account- <yjchen>

 

Attached Files
file #2109:  Test.h added by gcasa (614B - text/plain - Stress test for the class parser...)
file #1643:  HeaderParser.tar.gz added by yjchen (5KiB - application/x-gzip)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 5 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Sat 27 Nov 2004 11:14:02 AM UTCgcasaStatusNone=>Fixed
  Open/ClosedOpen=>Closed
  Attached File-=>Added Test.h, #1920
Wed 11 Aug 2004 04:08:52 PM UTCyjchenAttached File-=>Added HeaderParser.tar.gz, #1576
Wed 11 Aug 2004 02:38:22 AM UTCgcasaAssigned toNone=>gcasa

Back to the top


Powered by Savane 3.1-cleanup1