From: Florian Achleitner <florian.achleitner.2.6.31@gmail.com>
Date: Tue, 21 Aug 2012 09:44:55 GMT
Subject: Re: [PATCH v5 15/16] remote-svn: add marks-file regeneration
Message-ID: <1916249.rjj3ZYveKT@flomedio>
In-Reply-To: <7va9xpgngk.fsf@alter.siamese.dyndns.org>

On Monday 20 August 2012 16:20:27 Junio C Hamano wrote:
> Junio C Hamano <gitster@pobox.com> writes:
> > I think you meant something like:
> > 	init_notes(NULL, notes_ref, NULL, 0);
> > 	
> >         marksfile = fopen(marksfilename, "r");
> >         if (!marksfile) {
> >         
> >         	regenerate_marks(marksfilename);
> >         	
> >                 marksfile = fopen(marksfilename, "r");
> >                 if (!marksfile)
> >                 
> > 	                die("cannot read marks file!");
> > 	
> > 	} else {
> > 	
> >         	...
> > 
> > Also there is another call to regenerate_marks() without any
> > argument.  Has this even been compile-tested?

Yes it compiled and it works (is tested by t9020), but the compiler didn't 
complain because I left out void, so every argument was ok. I need to get used 
to that C-feature.

> 
> I've made regenerate_marks() to take (void) parameter list, as
> marksfilename is a file scope static and visible to everybody, and
> applied something like the above and queued the result in 'pu'.

That's exactly how I meant it. Thanks for your fixups!