diff support for the Eiffel language?

Hi!

After a longer pause, I did some programming in Eiffel again, and while doing so, why not use Git? It works!

However there's one little thing I noticed with "git diff": The conte4xt lines (staring with "@@") show the current function (in Perl and C), but they show the current "feature clause" in Eiffel (as opposed to the expected current feature). I wonder how hard it is to fix it (Observed in git 1.7.7 of openSUSE 12.1).

For the non-Eiffelists:

Eiffel has a class structure like this: class FOO

feature {BAR} -- This is a "feature clause", grouping related features (attributes/routines)

   baz (x,y : INTEGER) : STRING is
      -- blabla...
      do
         ...
      end -- baz

end -- class FOO

--- Now if I change something inside "baz", it would be nice if the @@-contect line would show "baz" (or more) instead of "feature {BAR}...".

Regards, Ulrich P.S. Apologies if the feature requested had been added already.

On Mon, Oct 22, 2012 at 1:58 PM, Ulrich Windl <Ulrich.Windl@rz.uni-regensburg.de> wrote:

See git.git's e90d065 for an example of adding a new diff pattern.

You could easily come up with a patch and send it to the list, however it would probably be good to CC some Eiffel language list in case there's some syntax oddities you've missed.

Am 10/22/2012 15:06, schrieb Ævar Arnfjörð Bjarmason:

It's not necessary to wait until there is built-in support for a new language.

For example, for Windows resource files, I have

*.rc    diff=winres

in .gitattributes or .git/info/attributes and

[diff "winres"]
        xfuncname =
"!^(BEGIN|END|FONT|CAPTION|STYLE)\n^[a-zA-Z_][a-zA-Z_0-9]*.*\n^[[:space:]]*([[:alnum:]_]+,
*DIALOG.*)"

in .git/config (the xfuncname is all on a single line). The first part beginning at ! up to \n tells to ignore the specified matches. The other parts separated by \n tell the things to put in the hunk header. You can have "ignore" parts (with exlamation mark) and "take this" parts (without) in any order that is convenient, as long as the last one is "take this".

-- Hannes

Hi!

I managed to have some success, although it's not perfect. Unfortunately the Eiffel grammar does not allow a 100% solution. Example:

---
local
  x : BOOLEAN
---
declares a local variable named `x', while

---
  x : BOOLEAN
   do ... end
---

Declares a function named `x'. Formal generic parameters like `x : ARRAY[ARRAY[STRING]]' make things more complicate. Anyway, here's my solution (probably incomplete): --- % git config --local -e [diff "Eiffel"] xfuncname = "!^[[:space:]]*(check|class|[Cc]reate|else|elseif|end|ensure|do|!|feature|from|indexing|inherit|is|local|loop|note|require|then|until)[[:space:]]*\n\ ^[[:space:]]*(class[[:space:]]*[[:alpha:]_][[:alnum:]_]*)\n\ ^[[:space:]]*([[:alpha:]_][[:alnum:]_]*[[:space:]]*\\([^(\")]+:[^(\")]+\\)[[:space:]]*:[[:space:]*[[:alpha:]_][[:alnum:]_]*[[:space:]]*[^[:space:]\"]?[^\"=]*)[[:space:]]*$\n\

^[[:space:]]*([[:alpha:]_][[:alnum:]_]*[[:space:]]*\\([^(\")]+:[^(\")]+\\))[[:space:]]*$\n\

^[[:space:]]*([[:alpha:]_][[:alnum:]_]*[[:space:]]*:[[:space:]*[[:alpha:]_][[:alnum:]_]*[[:space:]]*[^[:space:]\"]?[^\"=]*)[[:space:]]*$\n\ ^[[:space:]]*([[:alpha:]_][[:alnum:]_]*)[[:space:]]*$" % cat .git/info/attributes *.e diff=Eiffel ---

Regards, Ulrich

Nachricht <50854B90.50807@viscovery.net>:

"!^(BEGIN|END|FONT|CAPTION|STYLE)\n^[a-zA-Z_][a-zA-Z_0-9]*.*\n^[[:space:]]*(