Skip to content

Fix #149, * might be referenced before assignment#188

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
paulober:fix-149-might-be-referenced-before-assignment
Jul 29, 2021
Merged

Fix #149, * might be referenced before assignment#188
astrogeco merged 1 commit intonasa:integration-candidatefrom
paulober:fix-149-might-be-referenced-before-assignment

Conversation

@paulober
Copy link
Contributor

Describe the contribution

Testing performed

  1. IDE stops throwing this warning after the change

Expected behavior changes

  • stop throwing an UnboundLocalError for some people

Contributor Info - All information REQUIRED for consideration of pull request
Paul Oberosler, Individual

@skliper skliper added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label Jul 28, 2021
@skliper skliper added this to the 3.0.0 milestone Jul 28, 2021
@astrogeco astrogeco changed the base branch from main to integration-candidate July 29, 2021 03:08
@astrogeco astrogeco merged commit 64f7b86 into nasa:integration-candidate Jul 29, 2021
astrogeco added a commit to nasa/cFS that referenced this pull request Jul 29, 2021
nasa/cFS-GroundSystem#188, * might be referenced before assignment
astrogeco added a commit to nasa/cFS that referenced this pull request Jul 29, 2021
Combines:

nasa/cFE#1701, v6.8.0-rc1+dev789
nasa/osal#1116, v5.1.0-rc1+dev578
nasa/cFS-GroundSystem#192, v2.2.0-rc1+dev58

Includes:

**cFE**

nasa/cFE#1699, correct return code check
nasa/cFE#1700, documentation for FS APIs that return OSAL codes
nasa/cFE#1695, Adding coverage tests for cfe_es_apps.c
nasa/cFE#1673, Add File Utility Functional Tests.
nasa/cFE#1711, Add custom epoch support to TIME UT
nasa/cFE#1720, Requirements update for Caelum
nasa/cFE#1721, Add null check to CFE_ES_TaskID_ToIndex.
nasa/cFE#1719, scrub command documentation
nasa/cFE#1715, Add time arithmetic functional tests
nasa/cFE#1704, update docs for CFE_FS_BackgroundFileDumpRequest
nasa/cFE#1706, correct return code mismatches

**osal**

nasa/osal#1114, Add unit test branch coverage

**cFS-GroundSystem**

nasa/cFS-GroundSystem#185, Update tlm for ES Blockstats/memstats and TBL HK
nasa/cFS-GroundSystem#188, * might be referenced before assignment

Co-authored-by: Jacob Hageman <[email protected]>
Co-authored-by: Joseph Hickey <[email protected]>
Co-authored-by: Alex Campbell <[email protected]>
Co-authored-by: Jose F Martinez Pedraza <[email protected]>
Co-authored-by: Niall Mullane <[email protected]>
Co-authored-by: Paul <[email protected]>
@astrogeco astrogeco added CCB:Approved Indicates approval by CCB and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels Aug 4, 2021
@astrogeco
Copy link
Contributor

CCB:2021-08-04 APPROVED

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates approval by CCB community

Projects

None yet

Development

Successfully merging this pull request may close these issues.

UnboundLocalError: local variable 'itemStart' referenced before assignment

3 participants