Bugs item #3484469, was opened at 2012-02-04 16:23 Message generated for change (Comment added) made by eighthave You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=507079&aid=3484469...

Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None

Status: Closed

Resolution: None Priority: 5 Private: No Submitted By: Hans-Christoph Steiner (eighthave) Assigned to: Nobody/Anonymous (nobody) Summary: some suggestions for using error(), verbose(), etc.

Initial Comment: Since Pd now has the nice log level handling, here is a commit from Pd-extended 0.43.1 that can be taken as suggestions for how to use error() and verbose() to make the Gem debug messages more useful. Right now they are very verbose and in the '2 - normal' level, and can be confusing especially to newbies.

Here's the commit in question, you can get the diff file from it too: http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=revision&revi...

(and sorry I mistakenly committed to the old pd-gem svn, ignore that)

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave)

Date: 2012-02-12 09:16

Message: Gem produces lots of messages to the Pd window, which is useful. I think they could be much more useful if they were posted at the different levels that the new log levels in the Pd window gives us. Someone who knows Gem and uses it is much more qualified to determine whether any given message should be posted at a level fatal, error, normal, debug, or verbose. I was just hoping to spread the idea. What's included here are really just trivial changes to illustrate the idea of how logpost()/verbose()/error/etc. could be used in Gem. Its easier to write code than words to describe code, so they are intended to be read more than applied.

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig) Date: 2012-02-12 09:09

Message: unfortunately either your patch-file or sourceforge-dl is broken (at least the "zip" file seems to be html), and the diff from the svn does not apply to current "master". i'd be happily including the patch if you could provide one for current Gem.

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2012-02-11 07:12

Message: I fixed a name clash of Pd's error() and local var called 'error'

----------------------------------------------------------------------

Comment By: IOhannes m zmölnig (zmoelnig) Date: 2012-02-11 01:59

Message: please be a bit more verbose about "related fix", so it is not necessary to open yet another link and inspect the svn commit.

----------------------------------------------------------------------

Comment By: Hans-Christoph Steiner (eighthave) Date: 2012-02-10 10:22

Message: and a related fix: http://pure-data.svn.sourceforge.net/viewvc/pure-data?view=revision&revi...

----------------------------------------------------------------------

You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=507079&aid=3484469...