Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

should the screen attribute in Layout really be optional? #32

Open
tomjnixon opened this issue Apr 14, 2021 · 0 comments
Open

should the screen attribute in Layout really be optional? #32

tomjnixon opened this issue Apr 14, 2021 · 0 comments
Labels
question Further information is requested

Comments

@tomjnixon
Copy link
Member

In the EAR reference code, the screen is always present, but set to some default dimensions if not specified.

Handling the case where there is no screen (and skipping screen-related processing) might be desirable, but is non-standard -- should we get rid of the boost::optional wrapper (changing the API), or just set it to a default value and complain if it's reset to boost::none by the user?

@tomjnixon tomjnixon added the question Further information is requested label Apr 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant