<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">>><br>
>>>>> Hi Marc,<br>
>>>>>><br>
>>>>>> Hi,<br>
>>>>>>> can someone explain the goal of this method? AFAIK it's never called.<br>
>>>>>>><br>
>>>>>>><br>
>>>>>> I introduce this method inside PMA_DisplayResults class.<br>
>>>>>> As you mentioned, this method has used nowhere.<br>
>>>>>> I thought that this will useful for further improvements as well<br>
>>>>> generally<br>
>>>>>> getter setter methods use inside classes.<br>
>>>>>> But not sure it will definitely use in future.<br>
>>>>>><br>
>>>>>> It can be removed since it is not using at the moment.<br>
>>>>>> What should be better ?<br>
>>>>>><br>
>>>>>> Regards !<br>
>>>>><br>
>>>>> Chanaka,<br>
>>>>> I'm not sure, let's see opinions of others on this.<br>
>>>>><br>
>>>> I think, as to set the local PMA_CommonFunctions object of any class<br>
>>> using setCommonFunctions(), we would always need the existing single object<br>
>>> using PMA_CommonFunctions::getInstance().<br>
>>> And when we use local getCommonFunctions(), it already sets the<br>
>>> local PMA_CommonFunctions object to PMA_CommonFunctions::getInstance(). So<br>
>>> I see no particular utility of setCommonFunctions().<br>
>>><br>
>>>><br>
>>>> Hi Marc,<br>
>>>><br>
>>>> I introduce common setter method for set any property of that class.<br>
>>>> So I'll remove setCommonFunctions() function since the expected<br>
>>>> behavior can obtain from the common setter.<br>
>>>><br>
>>><br>
>> Hi Atul,<br>
>><br>
>> Seems that is the point.<br>
>> Since PMA_CommonFunctions is a singleton class, that method makes no sense.<br>
>> Anyway I'm going to remove it. :)<br>
><br>
> Hi Chanaka,<br>
> Ok :). Apart from PMA_DisplayResults(), similar setCommonFunctions() is<br>
> used in many other classes too, please remove it from there too.<br>
<br>
</div></div>Done.<br></blockquote><div> </div></div>Hi,<br><br clear="all">Ohh, I forgot that.<br>Thanks Marc, for doing it for me. :)<br><br>Regards !<br>-- <br>____________________________________<br><br>Chanaka Indrajith<br>
Bsc.Computer Engineering Undergraduate<br>Faculty of Engineering<br>University of Peradeniya<br>Sri Lanka<br>____________________________________<br><br>