Skip to content

Fix issue 3543: Explicitly added <climits> for C limits constants#3592

Merged
christophe-murphy merged 1 commit intoarrayfire:masterfrom
edwinsolisf:issue3543
Aug 30, 2024
Merged

Fix issue 3543: Explicitly added <climits> for C limits constants#3592
christophe-murphy merged 1 commit intoarrayfire:masterfrom
edwinsolisf:issue3543

Conversation

@edwinsolisf
Copy link
Contributor

The header included in math.hpp in all backends does not include depending on the standard library implementation. is required for the preprocessor defined limit constants.

Description

  • Is this a new feature or a bug fix?: Bug fix
  • Why these changes are necessary: fixes possible include errors when compiling with some compilers
  • Potential impact on specific hardware, software or backends: all backends
  • New functions and their functionality: None
  • Can this PR be backported to older versions?: Yes
  • Future changes not implemented in this PR: None
    Fixes: [Build] Error : missing lib #include <climits> #3543

Changes to Users

  • Additional options added to the build: None
  • No action required by the user

Checklist

  • Rebased on latest master
  • Code compiles
  • Tests pass
  • Functions added to unified API
  • Functions documented

@christophe-murphy christophe-murphy merged commit cb09bfc into arrayfire:master Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Build] Error : missing lib #include <climits>

2 participants