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

Group
 
 

bug #9955: Gorm parse outlets redundantly.

Submitter:  -Deleted Account- <yjchen>
Submitted:  Tue 10 Aug 2004 04:12:58 PM UTC
   
 
Category:  Gorm Severity:  3 - Normal
Item Group:  Bug Status:  Fixed
Privacy:  Public Assigned to:  gcasa
Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

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>
Group administrator
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>
Group administrator
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>
Group administrator
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>
Group administrator
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>
Group administrator
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>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

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

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

Only logged-in users can vote.

 

Follow 5 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2004-11-27 gcasa StatusNone Fixed
    Open/ClosedOpen Closed
    Attached File- Added Test.h, #1920
2004-08-11 yjchen Attached File- Added HeaderParser.tar.gz, #1576
2004-08-11 gcasa Assigned toNone gcasa

Back to the top

Powered by Savane 3.13-d3ae.
Corresponding source code