Re: [PATCH 1/1] The lambda function in run() is not needed.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 05, 2009 at 01:25:00PM +0200, Hans de Goede wrote:
> Hi,
>
> I'm afraid this is not going to work, as partedExceptionWindow
> needs anaconda as argument and parted won't pass that in, that
> was the whole purpose of the lamda (afaik)

There are three partedExceptionWindow functions.  one in text.py one in
cmdline.py and one in gui.py.  If you notice I change the call from
partedExceptionWindow to self.partedExceptionWindow.  and the
partedExceptionWindow function that is in text.py does not need the
anaconda object.

Now if the intention was to use the function from another file.  Than I
am mistaken.  Was that the intention?

Regards.

>
> Regards,
>
> Hans
>
>
> On 05/05/2009 01:14 PM, Joel Granados Moreno wrote:
>> ---
>>   text.py |    2 +-
>>   1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/text.py b/text.py
>> index 9133811..832c46c 100644
>> --- a/text.py
>> +++ b/text.py
>> @@ -671,7 +671,7 @@ class InstallInterface:
>>           anaconda.id.fsset.registerProgressWindow(self.progressWindow)
>>           anaconda.id.fsset.registerWaitWindow(self.waitWindow)
>>
>> -        parted.exception_set_handler(lambda exn: partedExceptionWindow(exn, anaconda))
>> +        parted.exception_set_handler(self.partedExceptionWindow)
>>
>>   	lastrc = INSTALL_OK
>>   	(step, instance) = anaconda.dispatch.currentStep()
>
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
Joel Andres Granados
Brno, Czech Republic, Red Hat.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux