headers
jamshark70 | 19 Dec 2011 03:21
Picon

SF.net SVN: quarks:[2127] dewdrop_lib

Revision: 2127
          http://quarks.svn.sourceforge.net/quarks/?rev=2127&view=rev
Author:   jamshark70
Date:     2011-12-19 02:21:41 +0000 (Mon, 19 Dec 2011)
Log Message:
-----------
Move Instr extension methods into ddwCommon

Modified Paths:
--------------
    dewdrop_lib/ddwVoicer/miditest.sc

Added Paths:
-----------
    dewdrop_lib/ddwCommon/Misc/Instr-ext.sc

Added: dewdrop_lib/ddwCommon/Misc/Instr-ext.sc
===================================================================
--- dewdrop_lib/ddwCommon/Misc/Instr-ext.sc	                        (rev 0)
+++ dewdrop_lib/ddwCommon/Misc/Instr-ext.sc	2011-12-19 02:21:41 UTC (rev 2127)
 <at>  <at>  -0,0 +1,37  <at>  <at> 
+// Used especially by miditest()
+// but also by FxPatch, so these need to be in common
+
++ Instr {
+	argsAndIndices {
+		var out;	
+		out = IdentityDictionary.new;
+		func.def.argNames.do({ |name, i|
+			out.put(name, i);
(Continue reading)

Permalink | Reply | 0 comments |
headers
James Harkins | 19 Dec 2011 04:45
Picon

Devs: Please enable "inline warnings"!!

Might I suggest that all devs should do the following?

LanguageConfig.postInlineWarnings = true;
LanguageConfig.store;

Twice within the last week, I've pulled and then found an inline warning during library compilation.

Sure, most users won't see it unless they change the default, but it's better to commit clean code wherever possible.

hjh

--
James Harkins /// dewdrop world
jamshark70@...
http://www.dewdrop-world.net

"Come said the Muse,
Sing me a song no poet has yet chanted,
Sing me the universal."  -- Whitman

blog: http://www.dewdrop-world.net/words
audio clips: http://www.dewdrop-world.net/audio
more audio: http://soundcloud.com/dewdrop_world/tracks

_______________________________________________
sc-dev mailing list

info (subscription, etc.): http://www.beast.bham.ac.uk/research/sc_mailing_lists.shtml
archive: http://www.listarc.bham.ac.uk/marchives/sc-dev/
search: http://www.listarc.bham.ac.uk/lists/sc-dev/search/
(Continue reading)

Permalink | Reply | 4 comments |
headers
James Harkins | 19 Dec 2011 04:47
Picon

[patch] fix PQ:removeValue inline warning

This patch appears to be innocuous, but I haven't reviewed the code carefully. Maybe there's a reason why
the variables have to be declared inside the while loop?

hjh

--
James Harkins /// dewdrop world
jamshark70@...
http://www.dewdrop-world.net

"Come said the Muse,
Sing me a song no poet has yet chanted,
Sing me the universal."  -- Whitman

blog: http://www.dewdrop-world.net/words
audio clips: http://www.dewdrop-world.net/audio
more audio: http://soundcloud.com/dewdrop_world/tracks
Attachment (0001-Remove-function-inlining-warning-from-PQ-removeValue.patch): application/octet-stream, 1104 bytes
Permalink | Reply | 1 comment |
headers
jamshark70 | 19 Dec 2011 04:55
Picon

SF.net SVN: quarks:[2128] LinuxExternal/JACK/SCJConnection.sc

Revision: 2128
          http://quarks.svn.sourceforge.net/quarks/?rev=2128&view=rev
Author:   jamshark70
Date:     2011-12-19 03:55:28 +0000 (Mon, 19 Dec 2011)
Log Message:
-----------
Whitespace cleanup

Modified Paths:
--------------
    LinuxExternal/JACK/SCJConnection.sc

Modified: LinuxExternal/JACK/SCJConnection.sc
===================================================================
--- LinuxExternal/JACK/SCJConnection.sc	2011-12-19 02:21:41 UTC (rev 2127)
+++ LinuxExternal/JACK/SCJConnection.sc	2011-12-19 03:55:28 UTC (rev 2128)
 <at>  <at>  -1,7 +1,7  <at>  <at> 
 SCJConnection { classvar <>alsadef, <>scdef, <>prepend, <allports, <connections, <properties;
 	var <srcchan, <deschan, <source, <destination;
 	classvar <>verbose = false;
-	
+
 	*initClass{
 		alsadef = ["system:","playback_","capture_"];
 		scdef = ["SuperCollider:","in_","out_"];
 <at>  <at>  -29,7 +29,7  <at>  <at> 
 		var pp, line1, line2;
 		connections = ();
 		pp = Pipe.new(prepend++"jack_lsp -c", "r");
-		line1 = pp.getLine;
(Continue reading)

Permalink | Reply | 0 comments |
headers
James Harkins | 19 Dec 2011 05:02
Picon

[jack quark] SCJConnection 'connections' data structure is wrong

SCJConnection assumes that a jack source may connect to one and only one destination:

connections.put( line1.asSymbol, line2.copyToEnd(3).asSymbol );

'connections' should really be a dictionary of (source -> [dest0, dest1, dest2...]).

OK to commit? (I'd do a helpfile fix before checking in.)

hjh

Index: SCJConnection.sc
===================================================================
--- SCJConnection.sc	(revision 2128)
+++ SCJConnection.sc	(working copy)
 <at>  <at>  -26,24 +26,20  <at>  <at> 
 	}

 	*getconnections{
-		var pp, line1, line2;
+		var pp, line1, line2, destArray;
 		connections = ();
 		pp = Pipe.new(prepend++"jack_lsp -c", "r");
 		line1 = pp.getLine;
 		while({line1.notNil}, {
-			line2 = pp.getLine;
-			if ( line2.notNil,
-				{
-					if ( line2.containsStringAt(0, "   "),
-						{
-							connections.put( line1.asSymbol, line2.copyToEnd(3).asSymbol );
(Continue reading)

Permalink | Reply | 1 comment |
headers
Tim Blechmann | 19 Dec 2011 09:19
Picon

SF.net Git: supercollider branch, master, updated. d484c338485312499b25dc99a254948ef03417dd

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "supercollider".

The branch, master has been updated
       via  d484c338485312499b25dc99a254948ef03417dd (commit)
      from  fdd29332b8c349000a7c07bed7df9257fd5ddb37 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit d484c338485312499b25dc99a254948ef03417dd
Author: James Harkins <jamshark70@...>
Date:   Mon Dec 19 11:43:10 2011 +0800

    Remove function-inlining warning from PQ:removeValue

    Signed-off-by: Tim Blechmann <tim@...>

diff --git a/SCClassLibrary/Common/Collections/PriorityQueue.sc b/SCClassLibrary/Common/Collections/PriorityQueue.sc
index aedc048..cb8d170 100644
--- a/SCClassLibrary/Common/Collections/PriorityQueue.sc
+++ b/SCClassLibrary/Common/Collections/PriorityQueue.sc
 <at>  <at>  -28,11 +28,11  <at>  <at>  PriorityQueue {
 	notEmpty { ^this.isEmpty.not }

 	removeValue {|value|
-		var newObject = PriorityQueue();
(Continue reading)

Permalink | Reply | 0 comments |
headers
Tim Blechmann | 19 Dec 2011 09:19
Favicon
Gravatar

Re: [patch] fix PQ:removeValue inline warning

> This patch appears to be innocuous, but I haven't reviewed the code
> carefully. Maybe there's a reason why the variables have to be declared
> inside the while loop?

ah, yes ... thanks ... i will enable inline warnings in future ;)

tim

_______________________________________________
sc-dev mailing list

info (subscription, etc.): http://www.beast.bham.ac.uk/research/sc_mailing_lists.shtml
archive: http://www.listarc.bham.ac.uk/marchives/sc-dev/
search: http://www.listarc.bham.ac.uk/lists/sc-dev/search/
Permalink | Reply | 0 comments |
headers
danstowell | 19 Dec 2011 09:47
Picon

SF.net SVN: quarks:[2129] MathLib/classes/SpherCoords

Revision: 2129
          http://quarks.svn.sourceforge.net/quarks/?rev=2129&view=rev
Author:   danstowell
Date:     2011-12-19 08:47:23 +0000 (Mon, 19 Dec 2011)
Log Message:
-----------
mathlib: SystemOverrides->SystemOverwrites

Added Paths:
-----------
    MathLib/classes/SpherCoords/SystemOverwrites/

Removed Paths:
-------------
    MathLib/classes/SpherCoords/SystemOverrides/

This was sent by the SourceForge.net collaborative development platform, the world's largest Open
Source development site.

_______________________________________________
sc-dev mailing list

info (subscription, etc.): http://www.beast.bham.ac.uk/research/sc_mailing_lists.shtml
archive: http://www.listarc.bham.ac.uk/marchives/sc-dev/
search: http://www.listarc.bham.ac.uk/lists/sc-dev/search/
Permalink | Reply | 0 comments |
headers
James Harkins | 19 Dec 2011 10:38
Picon

[crucial] rgb is deprecated

Hey felix,

OK to commit (I can do it from here if easier for you)? It avoids:

WARNING: Method SimpleNumber:-rgb is deprecated and will be removed. Use Color:*new255 instead.

hjh

Index: instrSupport.sc
===================================================================
--- instrSupport.sc	(revision 2107)
+++ instrSupport.sc	(working copy)
 <at>  <at>  -126,12 +126,12  <at>  <at> 
 		toSpec.maxval = this.map(linMaxVal);
 		^toSpec
 	}
-	color { ^rgb(125, 255, 205) }
+	color { ^Color.new255(125, 255, 205) }
 	background { ^Color.black }
 }
 + StaticSpec {
 	color { ^Color.black}
-	background { ^rgb(125, 255, 205) }
+	background { ^Color.new255(125, 255, 205) }
 }
 + NonControlSpec {
 	color { ^Color.black.alpha_(0.7) }
 <at>  <at>  -139,7 +139,7  <at>  <at> 
 }
(Continue reading)

Permalink | Reply | 3 comments |
headers
James Harkins | 19 Dec 2011 10:46
Picon

Re: Re: clock issue with new build

At Sun, 18 Dec 2011 21:35:33 +0100 (CET),
Daniel Mayer wrote:
> 
> 
> ah, so everyTempoClock.seconds refers to system time and this
> can be the time since SC startup (OSX) or a different system time
> on another platform - this is not clear from the TempoClock help file:
> 
> "TempoClock.new(tempo, beats, seconds, queueSize); ...
> ... seconds defaults to the current elapsed time since SuperCollider startup ..."
> 
> but actually it defaults to a current elapsed system time just after SuperCollider startup
> (which is the time since SC startup on OSX)

Ah... that looks like one of those help remarks written from the OSX perspective and never updated for other
platforms :)

> Is there are platform-independant method to estimate the time since
> SC startup (or instantiation of default TempoClock) if default TempoClock's
> tempo has been changed ?

The best I can think of is to add a new TempoClock during startup and keep its tempo always 1. Then (I think, not tested):

myConstantTempoClock.secs2beats(something.seconds)

hjh

--
James Harkins /// dewdrop world
jamshark70@...
(Continue reading)

Permalink | Reply | 0 comments |

Gmane