patchGNU Octave - Patches: patch #8441, Add support to "wav"...

 
 

patch #8441: Add support to "wav" extension in playaudio

Submitter:  Júlio Hoffimann Mendes <juliohm>
Submitted:  Fri 25 Apr 2014 04:42:46 PM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Wont Do Privacy:  Public
Assigned to:  None Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Thu 05 Feb 2015 04:40:36 PM UTC, comment #4: 

audioplayer, audioread and audiodevinfo has been merged to default with cset http://hg.savannah.gnu.org/hgweb/octave/rev/99522db5b911 and made some further improvements since then.

These new functions and classes will be part of the 4.0 release so I'll close this report because this patch isn't needed anymore.

Andreas Weber <andy1978>
Group Member
Fri 25 Apr 2014 11:02:14 PM UTC, comment #3: 

Should the patch be merged meanwhile?

Another issue with the current implementation is that normal users don't have permission to write on "/dev". An error message is raised such as:


sh: 1: cannot create /dev/audio: Permission denied


Júlio Hoffimann Mendes <juliohm>
Fri 25 Apr 2014 09:55:51 PM UTC, comment #2: 

Yes, we have much better and more Matlab-compatible audio handling in progress. It will be merged into a future release of Octave, either 4.0 or 4.2.

I'm not sure what the future of this playaudio function will be once we have audioread and audioplayer in Octave.

Mike Miller <mtmiller>
Group Member
Fri 25 Apr 2014 08:06:23 PM UTC, comment #1: 


> We need better handling of sound files though, preferably without system() calls nor redirections to "/dev/audio".


We have it, supposedly:

http://gsoc2013octaveaudio.blogspot.ca/

Jordi Gutiérrez Hermoso <jordigh>
Group Member
Fri 25 Apr 2014 04:42:46 PM UTC, original submission:  

This trivial patch is self-explanatory.

We need better handling of sound files though, preferably without system() calls nor redirections to "/dev/audio".

Júlio Hoffimann Mendes <juliohm>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #31248:  playaudio.patch added by juliohm (814B - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by andy1978 (Posted a comment)
  • -email is unavailable- added by mtmiller (Posted a comment)
  • -email is unavailable- added by jordigh (Posted a comment)
  • -email is unavailable- added by juliohm (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-02-05 andy1978 StatusNone Wont Do
        Open/ClosedOpen Closed
    2014-04-25 juliohm Attached File- Added playaudio.patch, #31248

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code