Re: [Emacs-diffs] /srv/bzr/emacs/trunk r109512: Inline functions to examine and change buffer overlays.

[Dmitry Antipov]

On 08/08/2012 07:45 PM, Stefan Monnier wrote:

> Can we pause for a moment?
> As I mentioned to you earlier, you should post such patches for review
> before installing them.
> And explain why you think they're good.
> E.g. replacing buffer->overlays_before by "buffer_get_overlays (buffer,
> OV_BEFORE)" is far from "obviously good" in my eyes.
> If you intend to make those field accesses into something more complex,
> then please say so and explain why.  If not, then please stick to
> "foo->bar", as agreed for the FGET case.

I think that it's reasonable to have just one chain of overlays per
buffer, much like the markers and intervals chains per buffer text.
And buffer_get_overlays(b, WHAT) is expected to simplify transition
to this possible change.

Dmitry