Skip to content

Update Sprite.cpp#97

Open
G-o-T-o wants to merge 4 commits intoCircuitMess:masterfrom
G-o-T-o:master
Open

Update Sprite.cpp#97
G-o-T-o wants to merge 4 commits intoCircuitMess:masterfrom
G-o-T-o:master

Conversation

@G-o-T-o
Copy link

@G-o-T-o G-o-T-o commented Aug 24, 2021

TODO's added regarding errors when compiling without 'build_flags = -w' option

@G-o-T-o
Update Sprite.cpp
9d28b75 
TODO's added regarding errors when compiling without 'build_flags = -w' option
@G-o-T-o
Copy link
Author

G-o-T-o commented Aug 24, 2021

Option 'build_flags = -w' in platformio.ini file resolves the issue, but nevertheless the errors should be fixed at the source.

@filipbudisa
Copy link
Member

filipbudisa commented Aug 24, 2021 •
edited
Loading

Thanks for pointing this out! A return *this; at the end of each of those functions will remove the warnings. Add those changes and I'll merge the PR
Filip

G-o-T-o added 2 commits August 24, 2021 13:44
@G-o-T-o
Added missing return values to methods that expect one
4f1f158 
Added missing return values to:
1) Sprite& Sprite::resize(uint width, uint height)
2) Sprite& Sprite::setTransparent(bool transparent)
3) Sprite& Sprite::setChroma(Color color)
@G-o-T-o
Missing return values in two methods
71fd150 
Missing return values in two methods:
1) UI\LowRamScreen.impl: virtual Screen& Screen::addChild(Element*)
2) UI\ScrollLayout.cpp: virtual ElementContainer& ScrollLayout::addChild(Element*)
@G-o-T-o
Copy link
Author

G-o-T-o commented Aug 24, 2021 •
edited
Loading

NOTE:
a) the fist two commits deal with no-return-value errors regarding Sprite.cpp file and they should be resolved by the second commit (as suggested by @filipbudisa in the previous comment).
b) the third commit is about a new issue: additionally found no-return-value errors in other two files (LowRamScreen.impl and ScrollLayout.cpp); I guess this commit had to belong to a new PR (but obviously it does not). So I suggest that in the next commit the newly found errors are fixed and then the PR is merged as a "package" and closed.

In order to resolve the newly found no-return-value errors, please confirm/answer to the following assumption/question:

In case of

  1. UI\LowRamScreen.impl: virtual Screen& Screen::addChild(Element*)

probably return *this; will do.

But what to return from

  1. UI\ScrollLayout.cpp: virtual ElementContainer& ScrollLayout::addChild(Element*) ?

@G-o-T-o
Added missing return values to methods that expect one
383bd47 
Added missing return values to:
1) src/UI/ScrollLayout.cpp: ElementContainer& ScrollLayout::addChild(Element* element)
2) src/UI/LowRamScreen.impl: Screen& Screen::addChild(Element* element)
@G-o-T-o
Copy link
Author

G-o-T-o commented Aug 30, 2021

For ElementContainer& ScrollLayout::addChild(Element*) I returned also *this as ScrollLayout class is derived from ElementContainer class.

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@G-o-T-o @filipbudisa