Re: [PATCH] userdiff: add support for Fountain documents

2015-07-29 Thread Zoë Blade
Hi again! Where's this at? Your last regex looks perfect to me: ^((\\.[^.]|(int|ext|est|int\\.?/ext|i/e)[. ]).*)$ Do you need anything else from me? Thanks, Zoë.-- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordom

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-22 Thread Zoë Blade
Hi again Junio! Yes, your more elegant and accurate regex sounds much better than the way I was trying it. :) Please, go ahead and use yours instead. Thank you for your help! Thanks, Zoë.-- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vge

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-21 Thread Junio C Hamano
Zoë Blade writes: H, the pattern has too many question marks to make a simple panda brain spin. > "^((\\.|((int|est|ext)?\\.?|i(nt)?\\.?/e(xt)?\\.?) ).+)$" it can start with a dot, or match "something" at the beginning, followed by a SP (is a tab allowed there instead of SP, I have to wond

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-21 Thread Zoë Blade
On 20 Jul 2015, at 22:17, Junio C Hamano wrote: >> +PATTERNS("fountain", >> "^((\\.|(([Ii][Nn][Tt]|[Ee][Ss][Tt]|[Ee][Xx][Tt])?\\.?|[Ii]([Nn][Tt])?\\.?/[Ee]([Xx][Tt])?\\.?) >> ).+)$", >> + "[^ \t-]+"), > > Wouldn't IPATTERN() be a better choice here? Good point, thank you! Much better: I

[PATCH] userdiff: add support for Fountain documents

2015-07-21 Thread Zoë Blade
Add support for Fountain, a plain text screenplay format. Git facilitates not just programming specifically, but creative writing in general, so it makes sense to also support other plain text documents besides source code. In the structure of a screenplay specifically, scenes are roughly analogo

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-20 Thread Junio C Hamano
Zoë Blade writes: > diff --git a/userdiff.c b/userdiff.c > index 2ccbee5..5a600d6 100644 > --- a/userdiff.c > +++ b/userdiff.c > @@ -35,6 +35,8 @@ IPATTERN("fortran", > * they would have been matched above as a variable anyway. */ > > "|[-+]?[0-9.]+([AaIiDdEeFfLlTtXx][Ss]?[-+]?[0

[PATCH] userdiff: add support for Fountain documents

2015-07-19 Thread Zoë Blade
Add support for Fountain, a plain text screenplay format. Git facilitates not just programming specifically, but creative writing in general, so it makes sense to also support other plain text documents besides source code. In the structure of a screenplay specifically, scenes are roughly analogo

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-19 Thread Zoë Blade
On 17 Jul 2015, at 23:43, Junio C Hamano wrote: > * Although uppercase is recommended for Scene Headings to increase > readability, it is not required. > > * A line beginning with any of the following, followed by either a > dot or a space, is considered a Scene Heading (unless the line is >

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Junio C Hamano
Junio C Hamano writes: > Zoë Blade writes: > >> More information about the Fountain format can be found on its >> official website, at http://fountain.io . So I visited there. >> +PATTERNS("fountain", "^((INT|EST|EXT)?\\.[A-Z0-9' -]+)$", >> + "[^ \t-]+"), After skimming http://fountain.i

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Junio C Hamano
Zoë Blade writes: > Add support for Fountain, a plain text screenplay format. Git > facilitates not just programming specifically, but creative writing > in general, so it makes sense to also support other plain text > documents besides source code. > > In the structure of a screenplay specific

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Johannes Schindelin
Hi Zoë, On 2015-07-17 16:03, Zoë Blade wrote: > On 17 Jul 2015, at 14:03, Johannes Schindelin > wrote: > >> Maybe you want to add a paragraph explaining a bit more about Fountain, or >> at least link to http://fountain.io/? >> >> In any case, you will need to sign off on your patch: >> >> >

[PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Zoë Blade
Add support for Fountain, a plain text screenplay format. Git facilitates not just programming specifically, but creative writing in general, so it makes sense to also support other plain text documents besides source code. In the structure of a screenplay specifically, scenes are roughly analogo

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Zoë Blade
On 17 Jul 2015, at 14:03, Johannes Schindelin wrote: > Maybe you want to add a paragraph explaining a bit more about Fountain, or at > least link to http://fountain.io/? > > In any case, you will need to sign off on your patch: > > > https://github.com/git/git/blob/v2.4.6/Documentation/Su

Re: [PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Johannes Schindelin
Hi Zoë, On 2015-07-17 13:59, Zoë Blade wrote: > Add support for Fountain, a plain text screenplay format. In the > structure of a screenplay, scenes are roughly analogous to functions, > in the sense that it makes your job slightly easier if you can see > which ones were changed in a given rang

[PATCH] userdiff: add support for Fountain documents

2015-07-17 Thread Zoë Blade
Add support for Fountain, a plain text screenplay format. In the structure of a screenplay, scenes are roughly analogous to functions, in the sense that it makes your job slightly easier if you can see which ones were changed in a given range of patches. --- Documentation/gitattributes.txt | 2 ++