CEDET

Hi Eric,

 > I touched most of the semantic files adding in missing ###autload
 > tokens, and also removed all autoloads from semantic-load.  I
 > checked in the Makefile that will build all this stuff.  I've
 > resisted in the past since it is autogenerated, but no-one seems to
 > use EDE so there is no benefit.  This particular change means that
 > semantic from CVS is useless without a Makefile.

That's fine with me!

Attached you will find a patch to Makefile, semantic-java.el and
semantic-texi.el.

In the Makefile, before running `semantic-hack-autoloads', I set
`find-file-hooks' to nil and, more important,
`find-file-suppress-same-file-warnings' to t, to suppress annoying
prompts to confirm over writing of semantic-al.el.  Maybe could you
adapt this change to Project.ede?

In semantic-java.el and semantic-texi.el I moved misplaced autoload
cookies.

On another subject, I noticed that
`semantic-repeat-parse-whole-stream' accept the argument DEPTH which
is never used here.  Maybe could we remove it now?

Sincerely,
David

>>> David Ponce <david@...> seems to think that:
>Hi Eric,
>
> > I touched most of the semantic files adding in missing ###autload
> > tokens, and also removed all autoloads from semantic-load.  I
> > checked in the Makefile that will build all this stuff.  I've
> > resisted in the past since it is autogenerated, but no-one seems to
> > use EDE so there is no benefit.  This particular change means that
> > semantic from CVS is useless without a Makefile.
>
>That's fine with me!
>
>Attached you will find a patch to Makefile, semantic-java.el and
>semantic-texi.el.
>
>In the Makefile, before running `semantic-hack-autoloads', I set
>`find-file-hooks' to nil and, more important,
>`find-file-suppress-same-file-warnings' to t, to suppress annoying
>prompts to confirm over writing of semantic-al.el.  Maybe could you
>adapt this change to Project.ede?

I'll put that into semantic-autogen.el like this:

(if noninteractive
    (progn
      ;; if the user is doing this non-interactivly, we need to set up
      ;; these conveniences.
      (add-to-list 'load-path ".")
      (setq find-file-hooks nil
	    find-file-suppress-same-file-warnings t)
      ))

which should be equivalent.

>In semantic-java.el and semantic-texi.el I moved misplaced autoload
>cookies.

Thanks, could you check those in.

>On another subject, I noticed that
>`semantic-repeat-parse-whole-stream' accept the argument DEPTH which
>is never used here.  Maybe could we remove it now?
  [ ... ]

That makes sense.  Depth is only used while lexing, so it certainly
doesn't need to be used there.

Thanks
Eric

-- 
          Eric Ludlam:                 zappo@..., eric@...
   Home: http://www.ultranet.com/~zappo            Siege: http://www.siege-engine.com
Emacs: http://cedet.sourceforge.net               GNU: http://www.gnu.org

> I'll put that into semantic-autogen.el like this:
>
> (if noninteractive
> (progn
> ;; if the user is doing this non-interactivly, we need to set up
> ;; these conveniences.
> (add-to-list 'load-path ".")
> (setq find-file-hooks nil
> find-file-suppress-same-file-warnings t)
> ))
>
> which should be equivalent.


OK.

>> In semantic-java.el and semantic-texi.el I moved misplaced autoload
>> cookies.
>
>
> Thanks, could you check those in.

Done.

> That makes sense. Depth is only used while lexing, so it certainly
> doesn't need to be used there.

I checked that change in too.

David