# Adding spell checker to GIT 8 messages from 2007-01-04 to 2007-01-26. Participants: Deepak Barua, Johannes Schindelin. Thread: https://gitlist.dev/t/6224 ## Deepak Barua, 2007-01-04 20:46 Subject: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/b5a19cd20701041246we052685hd700580df2cc120d%40mail.gmail.com ``` Hi All, I and sasikumar have designed and built a spell checker into the pre-commit hook, could someone please test it and give us your comments. Just add the file with same name pre-commit hook to .git/hooks directory in your git project and make it executable. Regards Deepak -- Code Code Code Away ``` ## Johannes Schindelin, 2007-01-05 20:49 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.63.0701052147070.22628%40wbgn013.biozentrum.uni-wuerzburg.de In-Reply-To: ``` Hi, On Fri, 5 Jan 2007, Deepak Barua wrote: > I and sasikumar have designed and built a spell checker into > the pre-commit hook, could someone please test it and give us your > comments. It's a rather big chunk of code, and it is in perl, so I'd rather put it in another file, but oh well. There are a few remaining problems, and I don't even know if the second can be solved: - you miss "//" comments, - if the diff is only about a small part of a comment, you do not spell check that - you do not spell check literal messages (i.e. printf("No such fiel!")) Ciao, Dscho ``` ## Deepak Barua, 2007-01-06 03:21 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/b5a19cd20701051921j69f1a85bo97e39db7d8401222%40mail.gmail.com In-Reply-To: ``` Hi Johannes, I spell check "//" comments in in find_ext() if($_[0] =~ /\.cc|\.java/) { $start_pattern = qr!\s/\*\s|\s//\s!; } and $chosen_pattern eq qq!\s//\s! ? $end_pattern=qq!\s\014\s! : "End Pattern Not Found" ; Also yes i didnt include to use spell checking in string literals , i thought that would be irregular and cumbersome. Could you please test the code then i can include it in a seperate file ..? Regards Deepak On 1/6/07, Johannes Schindelin wrote: > Hi, > > On Fri, 5 Jan 2007, Deepak Barua wrote: > > > I and sasikumar have designed and built a spell checker into > > the pre-commit hook, could someone please test it and give us your > > comments. > > It's a rather big chunk of code, and it is in perl, so I'd rather put it > in another file, but oh well. > > There are a few remaining problems, and I don't even know if the second > can be solved: > > - you miss "//" comments, > - if the diff is only about a small part of a comment, you do not spell > check that > - you do not spell check literal messages (i.e. printf("No such fiel!")) > > Ciao, > Dscho > > -- Code Code Code Away ``` ## Johannes Schindelin, 2007-01-16 16:10 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.63.0701161709570.22628%40wbgn013.biozentrum.uni-wuerzburg.de In-Reply-To: ``` Hi, On Sat, 6 Jan 2007, Deepak Barua wrote: > Hi Johannes, > I spell check "//" comments in > in find_ext() > > if($_[0] =~ /\.cc|\.java/) { > > $start_pattern = qr!\s/\*\s|\s//\s!; > } > > and > > $chosen_pattern eq qq!\s//\s! ? $end_pattern=qq!\s\014\s! : "End > Pattern Not Found" ; > > Also yes i didnt include to use spell checking in string literals , i > thought that would be irregular and cumbersome. > > Could you please test the code then i can include it in a seperate file ..? Sorry, I am a little bit stressed at the moment... And I don't have the required dependencies installed. Ciao, Dscho ``` ## Deepak Barua, 2007-01-16 17:34 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/b5a19cd20701160934s16b8717er69b950cff055a5ae%40mail.gmail.com In-Reply-To: ``` Hi Johannes, It's okay take your time and suggest any changes and teach me how to make this code into a proper patch. Regards Deepak On 1/16/07, Johannes Schindelin wrote: > Hi, > > On Sat, 6 Jan 2007, Deepak Barua wrote: > > > Hi Johannes, > > I spell check "//" comments in > > in find_ext() > > > > if($_[0] =~ /\.cc|\.java/) { > > > > $start_pattern = qr!\s/\*\s|\s//\s!; > > } > > > > and > > > > $chosen_pattern eq qq!\s//\s! ? $end_pattern=qq!\s\014\s! : "End > > Pattern Not Found" ; > > > > Also yes i didnt include to use spell checking in string literals , i > > thought that would be irregular and cumbersome. > > > > Could you please test the code then i can include it in a seperate file ..? > > Sorry, I am a little bit stressed at the moment... And I don't have the > required dependencies installed. > > Ciao, > Dscho > > -- Code Code Code Away ``` ## Deepak Barua, 2007-01-25 14:36 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/b5a19cd20701250636i6e3f474bhebb16020e918c155%40mail.gmail.com In-Reply-To: ``` Hi Johannes, What about the spell checker...? Regards Deepak On 1/16/07, Deepak Barua wrote: > Hi Johannes, > It's okay take your time and suggest any changes and > teach me how to make this code into a proper patch. > > Regards > Deepak > > On 1/16/07, Johannes Schindelin wrote: > > Hi, > > > > On Sat, 6 Jan 2007, Deepak Barua wrote: > > > > > Hi Johannes, > > > I spell check "//" comments in > > > in find_ext() > > > > > > if($_[0] =~ /\.cc|\.java/) { > > > > > > $start_pattern = qr!\s/\*\s|\s//\s!; > > > } > > > > > > and > > > > > > $chosen_pattern eq qq!\s//\s! ? $end_pattern=qq!\s\014\s! : "End > > > Pattern Not Found" ; > > > > > > Also yes i didnt include to use spell checking in string literals , i > > > thought that would be irregular and cumbersome. > > > > > > Could you please test the code then i can include it in a seperate file ..? > > > > Sorry, I am a little bit stressed at the moment... And I don't have the > > required dependencies installed. > > > > Ciao, > > Dscho > > > > > > > -- > Code Code Code Away > -- Code Code Code Away ``` ## Johannes Schindelin, 2007-01-25 16:46 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/Pine.LNX.4.63.0701251745520.22628%40wbgn013.biozentrum.uni-wuerzburg.de In-Reply-To: ``` Hi, On Thu, 25 Jan 2007, Deepak Barua wrote: > What about the spell checker...? Sorry, time is scarce these days. (And I can't do it on the machines I am right now working on, since I may not install perl packages.) Ciao, Dscho ``` ## Deepak Barua, 2007-01-26 07:22 Subject: Re: Adding spell checker to GIT Message-ID: URL: https://gitlist.dev/e/b5a19cd20701252322q632b9a30xaa0091e312197a4%40mail.gmail.com In-Reply-To: ``` Hi Johannes, cold you atleast tell me how to make it a patch so i can officially submit it and others can test it ...? Regards Deepak On 1/25/07, Johannes Schindelin wrote: > Hi, > > On Thu, 25 Jan 2007, Deepak Barua wrote: > > > What about the spell checker...? > > Sorry, time is scarce these days. (And I can't do it on the machines I am > right now working on, since I may not install perl packages.) > > Ciao, > Dscho > > -- Code Code Code Away ```