Secondary Overlays Bugfix
Edward Steere <[email protected]> Sat, 26 Mar 2016 16:49:43 +0200
| Newsgroups | gmane.emacs.cedet |
|---|---|
| Message-ID | <[email protected]> |
--Apple-Mail=_1A55B746-AEF3-4774-A9BC-5433381DF29B Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=utf-8 Hi All, This is patch 2/4. It's a fix for overlays from the decorate package. It looks like the interface to `semantic-tag-create-secondary-overlay' changed somewhere along the line. When tags are linked to the buffer, the overlays are removed and then re-added using the unlink/link hooks. `semantic=E2=80=94tag-link-secondary-overlays=E2=80=99 tried to fetch = the list of secondary overlays on a tag, remove them from the tag object and then re add each = of them in turn. However, it passed the tag and the overlay to the `semantic-tag-create-secondary-overlay=E2=80=99 which should instead = accept the tag and a link hook to run once the secondary overlay has been created. This produces several error messages similar to the following =E2=80=9CError wrong type argument: symbolp #<overlay =E2=80=A6" I=E2=80=99m unsure whether by making my change I=E2=80=99ve broken the = intended way for this to work =E2=80=94 i.e. that there are multiple overlays which ought to = be added one after the other =E2=80=94 but it does look like it wasn=E2=80=99t working = altogether without this fix. Kind regards, Edward Steere --Apple-Mail=_1A55B746-AEF3-4774-A9BC-5433381DF29B Content-Disposition: attachment; filename=0002-Fixed-an-error-in-decorate-for-secondary-overlays.patch Content-Type: application/octet-stream; name="0002-Fixed-an-error-in-decorate-for-secondary-overlays.patch" Content-Transfer-Encoding: quoted-printable =46rom=208106d88c3fb4ae3b26a55b521193d0960583cec5=20Mon=20Sep=2017=20= 00:00:00=202001=0AFrom:=20Edward=20Steere=20<[email protected]>=0A= Date:=20Fri,=2025=20Mar=202016=2012:30:44=20+0200=0ASubject:=20[PATCH=20= 2/4]=20Fixed=20an=20error=20in=20decorate=20for=20secondary=20overlays.=0A= =0AIt=20looks=20like=20the=20function=20= semantic-tag-create-secondary-overlay=20might=0Ahave=20once=20taken=20a=20= tag=20as=20well=20as=20the=20secondary=20overlay=20it=20should=20update,=0A= however=20in=20the=20current=20version=20of=20CEDET=20it=20accepts=20a=20= tag=20and=20the=20hook=20to=0Arun=20after=20adding=20the=20secondary=20= overlay.=20I'm=20not=20sure=20if=20we're=20now=0Amissing=20some=20= overlays=20with=20my=20fix,=20but=20it=20at=20least=20conforms=20to=20= the=20API=20of=0Athe=20function.=0A---=0A=20= lisp/cedet/semantic/decorate.el=20|=2010=20+++++-----=0A=201=20file=20= changed,=205=20insertions(+),=205=20deletions(-)=0A=0Adiff=20--git=20= a/lisp/cedet/semantic/decorate.el=20b/lisp/cedet/semantic/decorate.el=0A= index=205bd683f..f0ffe22=20100644=0A---=20= a/lisp/cedet/semantic/decorate.el=0A+++=20= b/lisp/cedet/semantic/decorate.el=0A@@=20-242,11=20+242,11=20@@=20from=20= them=20in=20TAG."=0A=20=20=20(let=20((ol=20= (semantic-tag-secondary-overlays=20tag)))=0A=20=20=20=20=20;;=20Wipe=20= out=20old=20values.=0A=20=20=20=20=20(semantic--tag-put-property=20tag=20= 'secondary-overlays=20nil)=0A-=20=20=20=20;;=20Run=20all=20the=20link=20= hooks.=0A-=20=20=20=20(while=20ol=0A-=20=20=20=20=20=20= (semantic-tag-create-secondary-overlay=20tag=20(car=20ol))=0A-=20=20=20=20= =20=20(setq=20ol=20(cdr=20ol)))=0A-=20=20=20=20))=0A+=20=20=20=20;;=20= Previously=20it=20looks=20like=20the=20signature=20of=0A+=20=20=20=20;;=20= `semantic-tag-create-secondary-overlay'=20accepted=20the=20secondary=0A+=20= =20=20=20;;=20overlay=20to=20be=20updated.=20Now,=20it=20doesn't.=20It=20= takes=20the=20link=20hook=0A+=20=20=20=20;;=20to=20run,=20so=20instead=20= we=20simply=20call=20it=20with=20tag.=0A+=20=20=20=20= (semantic-tag-create-secondary-overlay=20tag)))=0A=20=0A=20;;;=20= Secondary=20Overlay=20Uses=0A=20;;=0A--=20=0A2.5.4=20(Apple=20Git-61)=0A=0A= --Apple-Mail=_1A55B746-AEF3-4774-A9BC-5433381DF29B Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline ------------------------------------------------------------------------------ Transform Data into Opportunity. Accelerate data analysis in your applications with Intel Data Analytics Acceleration Library. Click to learn more. http://pubads.g.doubleclick.net/gampad/clk?id=278785351&iu=/4140 --Apple-Mail=_1A55B746-AEF3-4774-A9BC-5433381DF29B Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Cedet-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/cedet-devel --Apple-Mail=_1A55B746-AEF3-4774-A9BC-5433381DF29B--