This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Compile objc-lang.c, objc-exp.tab.c [1/5]



On Wednesday, February 19, 2003, at 12:06 PM, Elena Zannoni wrote:


Adam Fedor writes:
Now that I'm almost done, I finally understand the patch lingo (Thanks
Michael)! This is a repost of the first of five final patches for ObjC
suppport:

http://sources.redhat.com/ml/gdb-patches/2003-01/msg00049.html

The other two that I also need approval for are:

http://sources.redhat.com/ml/gdb-patches/2003-01/msg00050.html
http://sources.redhat.com/ml/gdb-patches/2003-01/msg00080.html

and two more (symtab.c, and linespec.c) will be coming soon.

---

A new year brings a final push to get Objective-C support in gdb. This
patch enables compilation and linking of the Objective-C files. I need
this now since the remaining patches require so many of the Objective-C
functions that it would be pointless to submit them otherwise.


2003-01-02 Adam Fedor <fedor at gnu dot org>

	* Makefile.in (COMMON_OBS): Add objc-lang.o
	(YYOBJ): Add objc-exp.tab.o
	(maint.o, utils.o): Add $(objc_lang_h)
	* maint.c (maintenance_demangle): Uncomment language_objc case.
	* utils.c (fprintf_symbol_filtered): Likewise.
	* parser-defs.h (start_msglist, add_msglist, end_msglist): Declare
	Objective-C parsing functions (from objc-lang.c).


I think this is just 'obvious' in a sense given the code has been
already approved, and it is just dormant.  However, do we have
testcases already? I think we should have a gdb.objc
subdirectory. (sorry, i don't remember if you posted them).  I would
feel uncomfortable enabling objc w/o having tests, even very simple
ones, it could bitrot fast.



Yes I have one. I was going to wait till all the patches were in, since, AFAIK most of the tests won't pass until all my patches are in.


How should I submit it? I suppose it would have to be a tar file.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]