Skip to content

Cleared compiler warnings except for Highs_compilationDate#2771

Merged
jajhall merged 3 commits intolatestfrom
latest-warnings
Feb 2, 2026
Merged

Cleared compiler warnings except for Highs_compilationDate#2771
jajhall merged 3 commits intolatestfrom
latest-warnings

Conversation

@jajhall
Copy link
Member

@jajhall jajhall commented Jan 23, 2026

I don't understand why there is a compiler warning for

static const char* Highs_compilationDate(void);

in /home/jajhall/HiGHS/highs/interfaces/highs_c_api.cpp, but it's deprecated

Other compiler warnings cleared (except for those from cuPDLP-c)

@jajhall jajhall requested a review from galabovaa January 23, 2026 17:31
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 90.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 80.32%. Comparing base (a058a69) to head (67d50c3).
⚠️ Report is 63 commits behind head on latest.

Files with missing lines Patch % Lines
highs/interfaces/highs_c_api.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           latest    #2771      +/-   ##
==========================================
- Coverage   80.32%   80.32%   -0.01%     
==========================================
  Files         348      348              
  Lines       86073    86095      +22     
==========================================
+ Hits        69141    69154      +13     
- Misses      16932    16941       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@jajhall jajhall merged commit 65dcfcb into latest Feb 2, 2026
393 of 402 checks passed
@jajhall jajhall deleted the latest-warnings branch February 2, 2026 19:43
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.

2 participants

Comments