Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [ALSA - driver 0001208]: 2 Typos and a little Question to device.c
@ 2005-06-23 10:14 bugtrack
  0 siblings, 0 replies; 4+ messages in thread
From: bugtrack @ 2005-06-23 10:14 UTC (permalink / raw)
  To: alsa-devel


A NOTE has been added to this issue.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1208> 
======================================================================
Reported By:                Henne
Assigned To:                
======================================================================
Project:                    ALSA - driver
Issue ID:                   1208
Category:                   1_OTHERS
Reproducibility:            always
Severity:                   text
Priority:                   normal
Status:                     new
Distribution:               
Kernel Version:             
======================================================================
Date Submitted:             06-22-2005 23:58 CEST
Last Modified:              06-23-2005 12:14 CEST
======================================================================
Summary:                    2 Typos and a little Question to device.c
Description: 
The comment @type of snd_device_new() is wrong. I belive it has to be
SNDDRV_DEV_XXX instead of SNDDRV_DEV_TYPE_XXX .

The comment in snd_device_disconnect() has to be snd_device_disconnect
instead of snd_device_free.

The sound_assert() macros in snd_device_new() and snd_device_register()
have a couple of expessions, and I belive it would be better for debuggin
if the macro calls would be called for paramter seperately, like in all
the other functions.

 
======================================================================

----------------------------------------------------------------------
 tiwai - 06-23-05 10:50 
----------------------------------------------------------------------
Agreed.  Patches are welcome :)
(See NEWS section for the detail how to write a patch.)

----------------------------------------------------------------------
 Henne - 06-23-05 12:14 
----------------------------------------------------------------------
Here we go!

Issue History
Date Modified  Username       Field                    Change              
======================================================================
06-22-05 23:58 Henne          New Issue                                    
06-23-05 10:50 tiwai          Note Added: 0005166                          
06-23-05 12:12 Henne          File Added: device-patch.diff                    
06-23-05 12:14 Henne          Note Added: 0005179                          
======================================================================




-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [ALSA - driver 0001208]: 2 Typos and a little Question to device.c
@ 2005-06-29 18:32 bugtrack
  0 siblings, 0 replies; 4+ messages in thread
From: bugtrack @ 2005-06-29 18:32 UTC (permalink / raw)
  To: alsa-devel


The following issue has been RESOLVED.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1208> 
======================================================================
Reported By:                Henne
Assigned To:                tiwai
======================================================================
Project:                    ALSA - driver
Issue ID:                   1208
Category:                   1_OTHERS
Reproducibility:            always
Severity:                   text
Priority:                   normal
Status:                     resolved
Distribution:               
Kernel Version:             
Resolution:                 fixed
Fixed in Version:           
======================================================================
Date Submitted:             06-22-2005 23:58 CEST
Last Modified:              06-29-2005 20:32 CEST
======================================================================
Summary:                    2 Typos and a little Question to device.c
Description: 
The comment @type of snd_device_new() is wrong. I belive it has to be
SNDDRV_DEV_XXX instead of SNDDRV_DEV_TYPE_XXX .

The comment in snd_device_disconnect() has to be snd_device_disconnect
instead of snd_device_free.

The sound_assert() macros in snd_device_new() and snd_device_register()
have a couple of expessions, and I belive it would be better for debuggin
if the macro calls would be called for paramter seperately, like in all
the other functions.

 
======================================================================

----------------------------------------------------------------------
 Henne - 06-28-05 16:28 
----------------------------------------------------------------------
Here we go!

Take the third patch(device_c.diff). Forgot to sign off.
Why can't I erase my submitted files?
I wrote a patch for core.h and add it to this thread.



----------------------------------------------------------------------
 tiwai - 06-29-05 20:32 
----------------------------------------------------------------------
Thanks, both patches are applied to CVS.

The restriction of removal of attachments is the feature of mantis.
Perhaps because there is no owner for attachments...

Issue History
Date Modified  Username       Field                    Change              
======================================================================
06-22-05 23:58 Henne          New Issue                                    
06-23-05 10:50 tiwai          Note Added: 0005166                          
06-23-05 12:12 Henne          File Added: device-patch.diff                    
06-23-05 12:14 Henne          Note Added: 0005179                          
06-23-05 12:17 Henne          File Added: device-patch2.diff                    
06-23-05 12:18 Henne          Note Edited: 0005179                         
06-28-05 16:24 Henne          File Added: device_c.diff                    
06-28-05 16:28 Henne          Note Edited: 0005179                         
06-28-05 16:28 Henne          File Added: core_h.diff                      
06-29-05 20:32 tiwai          Status                   new => resolved     
06-29-05 20:32 tiwai          Resolution               open => fixed       
06-29-05 20:32 tiwai          Assigned To               => tiwai           
06-29-05 20:32 tiwai          Note Added: 0005314                          
======================================================================




-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [ALSA - driver 0001208]: 2 Typos and a little Question to device.c
@ 2005-06-23  8:50 bugtrack
  0 siblings, 0 replies; 4+ messages in thread
From: bugtrack @ 2005-06-23  8:50 UTC (permalink / raw)
  To: alsa-devel


A NOTE has been added to this issue.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1208> 
======================================================================
Reported By:                Henne
Assigned To:                
======================================================================
Project:                    ALSA - driver
Issue ID:                   1208
Category:                   1_OTHERS
Reproducibility:            always
Severity:                   text
Priority:                   normal
Status:                     new
Distribution:               
Kernel Version:             
======================================================================
Date Submitted:             06-22-2005 23:58 CEST
Last Modified:              06-23-2005 10:50 CEST
======================================================================
Summary:                    2 Typos and a little Question to device.c
Description: 
The comment @type of snd_device_new() is wrong. I belive it has to be
SNDDRV_DEV_XXX instead of SNDDRV_DEV_TYPE_XXX .

The comment in snd_device_disconnect() has to be snd_device_disconnect
instead of snd_device_free.

The sound_assert() macros in snd_device_new() and snd_device_register()
have a couple of expessions, and I belive it would be better for debuggin
if the macro calls would be called for paramter seperately, like in all
the other functions.

 
======================================================================

----------------------------------------------------------------------
 tiwai - 06-23-05 10:50 
----------------------------------------------------------------------
Agreed.  Patches are welcome :)
(See NEWS section for the detail how to write a patch.)

Issue History
Date Modified  Username       Field                    Change              
======================================================================
06-22-05 23:58 Henne          New Issue                                    
06-23-05 10:50 tiwai          Note Added: 0005166                          
======================================================================




-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click

^ permalink raw reply	[flat|nested] 4+ messages in thread
* [ALSA - driver 0001208]: 2 Typos and a little Question to device.c
@ 2005-06-22 21:58 bugtrack
  0 siblings, 0 replies; 4+ messages in thread
From: bugtrack @ 2005-06-22 21:58 UTC (permalink / raw)
  To: alsa-devel


The following issue has been SUBMITTED.
======================================================================
<https://bugtrack.alsa-project.org/alsa-bug/view.php?id=1208> 
======================================================================
Reported By:                Henne
Assigned To:                
======================================================================
Project:                    ALSA - driver
Issue ID:                   1208
Category:                   1_OTHERS
Reproducibility:            always
Severity:                   text
Priority:                   normal
Status:                     new
Distribution:               
Kernel Version:             
======================================================================
Date Submitted:             06-22-2005 23:58 CEST
Last Modified:              06-22-2005 23:58 CEST
======================================================================
Summary:                    2 Typos and a little Question to device.c
Description: 
The comment @type of snd_device_new() is wrong. I belive it has to be
SNDDRV_DEV_XXX instead of SNDDRV_DEV_TYPE_XXX .

The comment in snd_device_disconnect() has to be snd_device_disconnect
instead of snd_device_free.

The sound_assert() macros in snd_device_new() and snd_device_register()
have a couple of expessions, and I belive it would be better for debuggin
if the macro calls would be called for paramter seperately, like in all
the other functions.

 
======================================================================

Issue History
Date Modified  Username       Field                    Change              
======================================================================
06-22-05 23:58 Henne          New Issue                                    
======================================================================




-------------------------------------------------------
SF.Net email is sponsored by: Discover Easy Linux Migration Strategies
from IBM. Find simple to follow Roadmaps, straightforward articles,
informative Webcasts and more! Get everything you need to get up to
speed, fast. http://ads.osdn.com/?ad_id=7477&alloc_id=16492&op=click

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2005-06-29 18:32 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-06-23 10:14 [ALSA - driver 0001208]: 2 Typos and a little Question to device.c bugtrack
  -- strict thread matches above, loose matches on Subject: below --
2005-06-29 18:32 bugtrack
2005-06-23  8:50 bugtrack
2005-06-22 21:58 bugtrack

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox