Skip to content

Conversation

@swift-kim
Copy link
Member

Do not print the "unimplemented method" warning when run with the --verbose-system-logs option. The method is always called when the Title widget rebuilds.

Copy link
Member

@JSUYA JSUYA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's an appropriate message because it's actually unimplemented.
Is there any reason why we should hide this?

@swift-kim
Copy link
Member Author

@JSUYA The Tizen platform doesn't support the functionality in the first place, so "unimplemented" is in fact misleading. It'd be better to make it just no-op.

Copy link
Member

@JSUYA JSUYA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

um.. If it is not supported, I think it may better print "unsupported".
Anyway. I got it.

@swift-kim
Copy link
Member Author

The API documentation says

Any part of the description that is unsupported on the current platform will be ignored.

so it should be safe to return Success instead of NotImplemented.

@swift-kim swift-kim merged commit 97c6cd1 into flutter-tizen:flutter-3.0.0-tizen Jul 7, 2022
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.

3 participants