Skip to content

Conversation

@oWnOIzRi
Copy link
Contributor

enable warnings in arithmetic



use strict;
use warnings;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me think about this for a bit. These are programs designed for end users who might not even know that Perl is the language behind them. As such, warnings aren't a good interface for them. Warnings are a good developers tool and we should ensure that the programs are warnings safe, but shipping end products with developer settings is a different beast.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've decided for the time being that we won't enable warnings. The main problem is that our audience isn't necessarily Perl programmers or even developers. Something be warnings safe now might not be with a future version. The intended user probably won't care or understand the warning.

We can test and develop with warnings on, but not ship with warnings.

@briandfoy briandfoy mentioned this pull request Oct 21, 2019
@briandfoy
Copy link
Owner

I'm tracking this, but I want to improve the tests to make sure everything happened correctly. That's just a limit on my time. I think the changes are good, but I also know that bugs don't care what I think. :)

@briandfoy briandfoy closed this Nov 9, 2019
@briandfoy briandfoy self-assigned this Jul 11, 2023
@briandfoy briandfoy added Status: released there is a new release with this fix Status: rejected this change is rejected Type: modernization updating programs to current practices Program: arithmetic The arithmetic program and removed Status: released there is a new release with this fix labels Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Program: arithmetic The arithmetic program Status: rejected this change is rejected Type: modernization updating programs to current practices

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants