Skip to content
This repository has been archived by the owner on Nov 18, 2023. It is now read-only.

move boilerplate from constructor to setup #53

Closed
umlaeute opened this issue Oct 30, 2019 · 1 comment
Closed

move boilerplate from constructor to setup #53

umlaeute opened this issue Oct 30, 2019 · 1 comment
Labels

Comments

@umlaeute
Copy link
Contributor

since you talked about doing a new release i thought it time to report this minor issue:

whenever an object from the library is instantiated, a nice boilerplate is displayed in the Pd-console:

[] part of PuREST JSON version
For more info see http://ix.residuum.org/pd/purest_json.html

now i'm a big fan of showing boilerplates and use them extensively in my own libraries.
however, i also see the concerns of those who feel like being drowned in unsolicited noise.

therefore, I'd like to suggest to not display the boilerplate whenever a new object ins instantiated, but instead reduce the number of shows to a single one.
this can most easily be achieved by moving the call to purest_json_lib_info() from the object constructor (e.g. rest_new()) to the setup function (e.g. rest_setup()).

an alternative is to use a static int firsttime=1; flag, but i don't see much benefits to do that.

@residuum residuum added the low label Oct 30, 2019
@umlaeute
Copy link
Contributor Author

umlaeute commented Nov 4, 2019

hmm, it seems with current HEAD (6e23dbd), the splash-screen is gone altogether.

you probably can close this then,

@residuum residuum closed this as completed Nov 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants