Re: [Libreoffice] [SCRIPT, EASYHACK] "add-modelines", a script to add modelines to source files

Hi Caolán,

The EasyHack wasn't clear on whether the script should wipe out any
existing modelines. In the interests of caution, I had it back off if
they were present. However, I can see that overwritting existing
modelines is needed, especially if the script is to be at all useful
after it's first run... ;) I think this feature calls for a script
getopts-style option where one can select to either not clobber exsiting
modelines (the default) or remove and replace existing modelines.

I'll churn on that for a bit and see if I can make it work. Thanks for
the feedback. :)

Cheers,

Jesse Adelman
Bold and Busted LLC
San Francisco, CA
http://www.boldandbusted.com/

On Sat, 09 Oct 2010 20:24 +0100, "Caolán McNamara" <caolanm@redhat.com>
wrote:
> On Sat, 2010-10-09 at 11:41 -0700, someone@boldandbusted.com wrote:
> > Howdy, all. It took some effort for this SysAdmin to craft a shell
> > script to meet this "EasyHack",
>
> The principal looks good. Here's a edge-case test file though, e.g.
>
> writer/sw/source/filter/ww8/ww8atr.cxx
>
> it already has a "/* vi:"... modeline at the bottom, so maybe best thing
> is to detect and remove conflicting modelines. Though there's so few of
> them, they could be handled manually, so its no biggy.
>
> C.