Remove unused variable from Alphametics stub

[coriolinus]

Travis fails in general right now, because we have an unused variable in the Alphametics stub. This is causing #381 to fail right now, and possibly others.

This simply removes the unused name from the Alphametics stub. This changes no interfaces or tests; students may rename the variable to anything they prefer. It simply ensures that the _test/ensure-stubs-compile.sh test actually passes.

[petertseng]

I support this change.

I strongly believe this is not the reason #381 fails, if #381 preserved existing behaviour.

Stubs are never compiled with [deny(warnings)] in current tests. If #381 fails, then #381 did not preserve that behaviour and instead represents a change in behaviour.

[coriolinus]

I need to update #381. Your comment about the deny warnings insertion script line makes sense, which means that I did introduce an unintentional change in behavior there.

…

On Wed, Nov 8, 2017 at 1:16 AM, Peter Tseng ***@***.***> wrote: I support this change. I strongly believe this is not the reason #381 <#381> fails, if #381 <#381> preserved existing behaviour. Stubs are never compiled with [deny(warnings)] in current tests. If #381 <#381> fails, then #381 <#381> did not preserve that behaviour and instead represents a change in behaviour. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#388 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHdeTj1OtMrdozHEUvd7YunDxeae1cpQks5s0PLggaJpZM4QUna7> .