-
Notifications
You must be signed in to change notification settings - Fork 260
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CCOLAMD] fix set but unused variable Int cs
#39
Conversation
@DrTimothyAldenDavis do you mind taking a look at this? |
Yes, but it's later in my queue. Tight deadlines for other critical work
at the moment.
…On Thu, May 21, 2020 at 11:01 AM acxz ***@***.***> wrote:
@DrTimothyAldenDavis <https://github.com/DrTimothyAldenDavis> do you mind
taking a look at this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEYIIOKYQ7XAMTJAAAXIB6TRSVF6NANCNFSM4M5PDOQA>
.
|
Also need the signed contributor agreement
On Thu, May 21, 2020 at 12:25 PM Tim Davis <[email protected]>
wrote:
… Yes, but it's later in my queue. Tight deadlines for other critical work
at the moment.
On Thu, May 21, 2020 at 11:01 AM acxz ***@***.***> wrote:
> @DrTimothyAldenDavis <https://github.com/DrTimothyAldenDavis> do you
> mind taking a look at this?
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#39 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AEYIIOKYQ7XAMTJAAAXIB6TRSVF6NANCNFSM4M5PDOQA>
> .
>
|
Thx for responding! No worries. Didn't realize there was a CLA, mb. I would recommend looking at cla-bot or cla-assisstant to automate this process. Would this work as a submission of the signed agreement based on:
|
Thanks for the heads up. I'll check out the cla-bot or cla-assistant.
…On Thu, May 21, 2020 at 1:10 PM acxz ***@***.***> wrote:
Thx for responding!
No worries.
Didn't realize there was a CLA, mb. I would recommend looking at cla-bot
<https://github.com/apps/cla-bot> or cla-assisstant
<https://github.com/cla-assistant/cla-assistant> to automate this process.
Would this work as a submission of the signed agreement based on:
https://github.com/DrTimothyAldenDavis/SuiteSparse/blob/master/CONTRIBUTOR-LICENSE.txt#L29
?
The GitHub user, X acxz , has read and understood the Contributor License
Agreement located at
https://github.com/DrTimothyAldenDavis/SuiteSparse/blob/master/CONTRIBUTOR-LICENSE.txt
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEYIIOPCMTYMICKDHZPXYCDRSVVALANCNFSM4M5PDOQA>
.
|
The variable is used when debugging is enabled. Deleting it breaks the code in that case. |
Resolves #38