Skip to content

Add deprecation message to polynomial pointer, order constructor - #974

Merged
mborland merged 1 commit into
boostorg:developfrom
mborland:156
Sep 1, 2026
Merged

mborland merged 1 commit into
boostorg:developfrom
mborland:156

Conversation

@mborland

@mborland mborland commented Apr 6, 2023

Copy link
Copy Markdown
Member

Now that we require C++14 this is an easy fix @NAThompson.
Closes #156

@mborland mborland changed the title Add deprecation message to pointer, order constructor Add deprecation message to polynomial pointer, order constructor Apr 6, 2023
@mborland mborland mentioned this pull request Apr 7, 2023
@mborland mborland closed this Apr 13, 2023
@mborland mborland reopened this Apr 13, 2023
@NAThompson

Copy link
Copy Markdown
Collaborator

@mborland : Any reason this didn't get merged?

@mborland

Copy link
Copy Markdown
Member Author

@mborland : Any reason this didn't get merged?

Some of the old compilers don't support deprecated messages on classes if I remember correctly.

@NAThompson

Copy link
Copy Markdown
Collaborator

@mborland : Have we dropped support for the old compilers yet?

@mborland mborland closed this Sep 1, 2026
@mborland mborland reopened this Sep 1, 2026
@mborland

mborland commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

@mborland : Have we dropped support for the old compilers yet?

I think so. I'll have to rebase this onto develop to confirm

@codecov

codecov Bot commented Sep 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.41%. Comparing base (3c1c431) to head (74e3386).
⚠️ Report is 4 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #974   +/-   ##
========================================
  Coverage    95.40%   95.41%           
========================================
  Files          831      831           
  Lines        69540    69550   +10     
========================================
+ Hits         66346    66360   +14     
+ Misses        3194     3190    -4     
Files with missing lines Coverage Δ
include/boost/math/tools/polynomial.hpp 99.27% <ø> (ø)

... and 3 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3c1c431...74e3386. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mborland
mborland merged commit 156df0b into boostorg:develop Sep 1, 2026
77 checks passed
@mborland
mborland deleted the 156 branch September 1, 2026 17:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Heap buffer overflow in Polynomial:

2 participants