residuum / PuRestJson

PuREST JSON is a library for connecting Puredata (Pd) to HTTP services and encoding and decoding JSON data.
ix.residuum.org/pd/purest_json.html
Other
74 stars 9 forks source link

move boilerplate from constructor to setup #53

Closed umlaeute closed 4 years ago

umlaeute commented 4 years ago

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.

umlaeute commented 4 years ago

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

you probably can close this then,