Re: ConstantValueTag patch

[Laurie Hendren <Laurie.Hendren@comlab.ox.ac.uk>]

Yes, we should add them.  I'll ask Jennifer to do it,  we also need
them for a current project.

Thanks, Laurie


+----------------------------------------------------------------+
| Laurie Hendren, Professor, Computer Science, McGill University |
| Visiting Fellow, Oxford University Computing Laboratory,       |
| Wolfson Building, Parks Road         Fax: +44 (0) 1865 273839  |
| Oxford, UK  OX1 3QD    Tel: +44 1865 283551, 01865 283551 (UK) |
| hendren@cs.mcgill.ca       http://www.sable.mcgill.ca/~hendren |
+----------------------------------------------------------------+

On Tue, 11 May 2004, Archie Cobbs wrote:

> Archie Cobbs wrote:
> > I've got a patch that adds support for "ConstantValueTag" tags in Soot.
> > These tags are associated with SootField's that are initialized with
> > constant values in the class files. Right now in Soot this information
> > is simply lost; this patch tries to address that problem.
> >
> > This patch is part of a larger patch that also adds "SourceFileTag"
> > and "InnerClassTag" tags (also information currently lost with Soot),
> > plus a couple of other minor fixes (some of which are already in CVS;
> > this patch is against version 2.1.0).
> >
> > Previously this patch was deemed not suitable because it only adds
> > support for reading these in, but does not add support to the jimple
> > language, parser, etc. for writing them out.
> >
> > However, I'd like to request again that they (or some equivalent) be
> > added, on the simple principle that 1/2 of an implementation is better
> > than no implementation at all, and it doesn't look like a complete
> > implementation is in the works by anyone.
> >
> > Moreover, I'm guessing these new tags would be useful to other folks
> > than just me. Do other folks have objections to adding them?
>
> I didn't hear any objections to committing these patches. Could
> some Soot maintainer(s) give some feedback?
>
> Thanks,
> -Archie
>
> __________________________________________________________________________
> Archie Cobbs      *        CTO, Awarix        *      http://www.awarix.com
>