On Thu, Oct 20, 2011 at 11:11:48AM +0100, Guido Trotter wrote: > On Thu, Oct 20, 2011 at 11:10 AM, Iustin Pop <ius...@google.com> wrote: > > On Thu, Oct 20, 2011 at 11:10:29AM +0100, Guido Trotter wrote: > >> On Thu, Oct 20, 2011 at 10:54 AM, Iustin Pop <ius...@google.com> wrote: > >> > >> > Since in patch 3/3 you anyway call this function twice, just make it > >> > return both sets of files. Default arguments are not nice. > >> > > >> > >> So return ([all], [optional]) ? Ok > > > > ack, thanks! > > > > Note that I'll have to merge 2 and 3 then, or the call will break in between > :)
Or alternatively modify 2 to change the call from files = GetAncilarry() to (files, _) = GetAncilarry(). Either ways is fine. thanks! iustin
