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

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



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 redhat com
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

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


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