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" <caol...@redhat.com> wrote: > On Sat, 2010-10-09 at 11:41 -0700, some...@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. > > _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
