On Thu, 2010-12-16 at 14:22 -0800, Jakub Narebski wrote:
> Sylvain Rabot <sylvain@xxxxxxxxxxxxxx> writes:
>
> > added: sql, php5, phps, bash, zsh, mk
> >
>
> There is one issue with this patch (see below), but otherwise I like
> it.
>
> > Signed-off-by: Sylvain Rabot <sylvain@xxxxxxxxxxxxxx>
> > ---
> > gitweb/gitweb.perl | 7 ++++---
> > 1 files changed, 4 insertions(+), 3 deletions(-)
> >
> > diff --git a/gitweb/gitweb.perl b/gitweb/gitweb.perl
> > index d521c93..db18d06 100755
> > --- a/gitweb/gitweb.perl
> > +++ b/gitweb/gitweb.perl
> > @@ -250,13 +250,14 @@ our %highlight_ext = (
> > # main extensions, defining name of syntax;
> > # see files in /usr/share/highlight/langDefs/ directory
> > map { $_ => $_ }
> > - qw(py c cpp rb java css php sh pl js tex bib xml awk bat ini spec tcl),
> > + qw(py c cpp rb java css php pl js tex bib xml awk bat ini spec tcl sql),
>
> Why did you remove 'sh' => 'sh' mapping? Shouldn't it be
>
> - qw(py c cpp rb java css php sh pl js tex bib xml awk bat ini spec tcl),
> + qw(py c cpp rb java css php sh pl js tex bib xml awk bat ini spec tcl sql),
>
I removed it on purpose but should have not, I was thinking
the mapping made below would be enough but no, you're right.
> > # alternate extensions, see /etc/highlight/filetypes.conf
> > 'h' => 'c',
> > + map { $_ => 'sh' } qw(bash zsh),
>
> Good idea.
Does ksh, csh can be highlighted as sh too ?
>
> > map { $_ => 'cpp' } qw(cxx c++ cc),
> > - map { $_ => 'php' } qw(php3 php4),
> > + map { $_ => 'php' } qw(php3 php4 php5 phps),
>
> Good idea.
>
> > map { $_ => 'pl' } qw(perl pm), # perhaps also 'cgi'
> > - 'mak' => 'make',
> > + map { $_ => 'make'} qw(mak mk),
>
> Good idea.
>
> Sidenote: git itself uses *.mak extension. Is *.mk popular?
Don't know but mk is mapped to Makefiles in the gtksourceview lib so I
added it here.
>
> > map { $_ => 'xml' } qw(xhtml html htm),
> > );
> >
> > --
> > 1.7.3.2
> >
>
--
Sylvain Rabot <sylvain@xxxxxxxxxxxxxx>
Attachment:
signature.asc
Description: This is a digitally signed message part