-----Original Message-----
From: John Baldwin <[email protected]>
Date: 2022-04-08, Friday at 12:27
To: Ravi Pokala <[email protected]>, <[email protected]>, 
<[email protected]>, <[email protected]>
Subject: Re: git: 885db9301ef9 - main - qlnx: Comment out remaining references 
to tx_data_bd.

    On 4/7/22 6:40 PM, Ravi Pokala wrote:
    > -----Original Message-----
    > From: <[email protected]> on behalf of John Baldwin 
<[email protected]>
    > Date: 2022-04-07, Thursday at 17:03
    > To: <[email protected]>, <[email protected]>, 
<[email protected]>
    > Subject: git: 885db9301ef9 - main - qlnx: Comment out remaining 
references to tx_data_bd.
    > 
    >      The branch main has been updated by jhb:
    > 
    >      URL: 
https://cgit.FreeBSD.org/src/commit/?id=885db9301ef9dfe420eff3c09aca497b7fc60924
    > 
    >      commit 885db9301ef9dfe420eff3c09aca497b7fc60924
    >      Author:     John Baldwin <[email protected]>
    >      AuthorDate: 2022-04-08 00:01:27 +0000
    >      Commit:     John Baldwin <[email protected]>
    >      CommitDate: 2022-04-08 00:01:27 +0000
    > 
    >          qlnx: Comment out remaining references to tx_data_bd.
    > 
    > Why comment out, as opposed to removing outright (like most of the other 
unused-but-set cleanup)?
    > 
    > Thanks,
    > 
    > Ravi (rpokala@)

    In general I have commented out code for variables if the only uses of 
those variables
    are in commented out code vs removing it (as one would need to also remove 
the commented
    out code to remove it entirely) in case someone wanted to resurrect the 
commented out
    code.

Ah! That makes sense.

Thanks,

Ravi (rpokala@)

    -- 
    John Baldwin

Reply via email to