Skip to content

Group standard library includes in AampDRMLicManager.cpp#967

Draft
Copilot wants to merge 2 commits intofeature/VPLAY-12576from
copilot/sub-pr-961-again
Draft

Group standard library includes in AampDRMLicManager.cpp#967
Copilot wants to merge 2 commits intofeature/VPLAY-12576from
copilot/sub-pr-961-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

Addresses code review feedback on PR #961 to follow standard include grouping conventions.

Changes

  • Moved <chrono> include from line 38 to line 28, adjacent to <pthread.h>
  • Removed extraneous blank lines around the include directive

Before/After

// Before: <chrono> isolated with blank lines
#include "PlayerSecInterface.h"

#include <chrono>

#include "AAMPAnomalyMessageType.h"

// After: Standard library headers grouped together
#include "DrmHelper.h"
#include <chrono>
#include <pthread.h>
#include "downloader/AampCurlStore.h"

Standard library headers are now grouped together as per C++ Core Guidelines, with project headers separated into their own sections.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Move <chrono> include to be adjacent to <pthread.h> and remove extra blank lines, following standard practice of grouping standard library headers together.

Co-authored-by: rajat698 <67538489+rajat698@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on precision for AAMP totalPerformRequest Group standard library includes in AampDRMLicManager.cpp Feb 4, 2026
Copilot AI requested a review from rajat698 February 4, 2026 15:24
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