Remove unused curve_distance_epsilon by iccir · Pull Request #32287 · matplotlib/matplotlib · GitHub
Skip to content

Remove unused curve_distance_epsilon - #32287

Merged
scottshambaugh merged 1 commit into
matplotlib:mainfrom
iccir:remove-curve-distance-epsilon
Sep 6, 2026
Merged

Remove unused curve_distance_epsilon#32287
scottshambaugh merged 1 commit into
matplotlib:mainfrom
iccir:remove-curve-distance-epsilon

Conversation

@iccir

@iccir iccir commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

PR summary

This PR removes curve_distance_epsilon from "src_agg_curves.cpp". As far as I can tell, this was present in the original agg 2.4 source but was never used.

It has generated a warning for at least the last decade, if not longer:

[7/33] Compiling C++ object extern/agg24-svn/libagg.a.p/src_agg_curves.cpp.o
../../extern/agg24-svn/src/agg_curves.cpp:24:18:
warning: unused variable 'curve_distance_epsilon' [-Wunused-const-variable]
    const double curve_distance_epsilon                  = 1e-30;

AI Disclosure

No AI used.

PR quality check

  • Use an expressive title, e.g. "Fix title font property precedence"
  • New and changed code is tested
  • [N/A] Plotting related features are demonstrated in an example
  • [N/A] New features and API changes have release notes
  • [N/A] Documentation complies with general and docstring guidelines

@scottshambaugh
scottshambaugh merged commit 6db85a3 into matplotlib:main Sep 6, 2026
41 checks passed
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.

3 participants