[petsc-dev] #if 0 stuff in source code
Karl Rupp
rupp at iue.tuwien.ac.at
Wed Jun 25 09:49:51 CDT 2014
Hey,
> I noticed the commented out code as well and wasn't sure whether it
> could be removed. I agree with Karl, it can go.
>
> Karl, I can deal with this in a new branch to not confuse things with
> the current pull request. I think this will merge rather easily down the
> road.
fine with me - you've got a better overview of the changes you applied.
I assume that haven't got any changes in the "#if 0"-code sections
anyway ;-)
Best regards,
Karli
>
> On 06/25/2014 01:33 AM, Karl Rupp wrote:
>> Hi,
>>
>> > There is a bunch of #if 0 droplets in the source code related
>> to the use of CUSP. Can all that be removed? It isn’t good to have
>> dead code like that in source. It’s ugly and confusing.
>>
>> Most of this is in the code base for at least 1.5 years now (it was
>> there when I first looked at it), so I think it can be removed as it
>> hasn't been needed in any way. However, I'd like to deal with
>> Dominic's pull request first in order not to make the merge more
>> painful than necessary.
>>
>> Best regards,
>> Karli
>>
>
>
More information about the petsc-dev
mailing list