Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

Refer the following document to know AAMP Coding Guidelines.

View file
nameAampCodingGuidelines.docx
height250

Dashboard link for current Sprint progress : https://ccp.sys.comcast.net/Dashboard.jspa?selectPageId=72972 

  • When you start work on ticket , move ticket status to Code development . Update Development Owner and Development Manager.  Ensure Sprint label is added.  For April/May Sprint its aamp_2004
  • When closing ticket (duplicate / or any reason) , update triage info with explanation and root cause analysis..
  • Please update ticket with development logs before sharing the review
  • Code review or Ticket must include clear test procedure for QA, not generic "Tested in Xi5/Xi6"
  • Whenever you raise any code review outside AAMP like AVE / XRE / Fog  , make sure you trigger verification build manually . Here is the link https://jenkins.ccp.xcal.tv/jenkins/job/Gerrit-Trigger-Custom-Verification-SingleReview-Pipeline/  . This will speed up merge process.

Check for compilation warnings - some can be dangerous.

Use proper printf format specifiers, i.e.

...

specifier

...

type

...

%d

...

Steps for using Coverity tool:

...