All of lore.kernel.org
 help / color / mirror / Atom feed
* Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils
@ 2008-11-21  2:50 Diego E. 'Flameeyes' Pettenò
  2008-11-21 11:52 ` Takashi Iwai
  0 siblings, 1 reply; 6+ messages in thread
From: Diego E. 'Flameeyes' Pettenò @ 2008-11-21  2:50 UTC (permalink / raw)
  To: alsa-devel


[-- Attachment #1.1: Type: text/plain, Size: 1366 bytes --]


I've taken some time to fix some autotools nitpicks and some generic
optimisations, I hope they can be useful:

git://git.flameeyes.eu/others/alsa/alsa-lib.git flameeyes

Diego E. 'Flameeyes' Pettenò (5):
      Don't use AC_CANONICAL_SYSTEM, only use AC_CANONICAL_HOST.
      Add the attributes.m4 macro file from xine/lscube.
      Check for --no-undefined linker flag and use it.
      Make sure that python libraries are passed through LIBADD.
      Make some static tables and strings constants.

git://git.flameeyes.eu/others/alsa/alsa-plugins.git flameeyes

Diego E. 'Flameeyes' Pettenò (6):
      Add the attributes.m4 macro file from xine/lscube.
      Check for --no-undefined linker flag and use it.
      Cleanup flags in maemo/Makefile.am
      [RFC] Don't use pow() for calculating a power of 2, use shift instead.
      Make some static tables and strings constants.
      Mark as static the functions not used outside their unit.

git://git.flameeyes.eu/others/alsa/alsa-utils.git flameeyes

Diego E. 'Flameeyes' Pettenò (4):
      Remove some dead code (comparisons between 0 and unsigned integers).
      Remove some unused variables.
      Mark static the functions not used outside their unit.
      Make some static tables and strings constants.


-- 
Diego "Flameeyes" Pettenò
http://blog.flameeyes.eu/


[-- Attachment #1.2: Type: application/pgp-signature, Size: 196 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

* Re: Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils
  2008-11-21  2:50 Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils Diego E. 'Flameeyes' Pettenò
@ 2008-11-21 11:52 ` Takashi Iwai
  2008-11-21 12:02   ` Diego E. 'Flameeyes' Pettenò
  2008-11-21 12:12   ` Diego E. 'Flameeyes' Pettenò
  0 siblings, 2 replies; 6+ messages in thread
From: Takashi Iwai @ 2008-11-21 11:52 UTC (permalink / raw)
  To: Diego E. 'Flameeyes' Pettenò; +Cc: alsa-devel

At Fri, 21 Nov 2008 03:50:32 +0100,
Diego E. 'Flameeyes' Pettenò wrote:
> 
> I've taken some time to fix some autotools nitpicks and some generic
> optimisations, I hope they can be useful:
> 
> git://git.flameeyes.eu/others/alsa/alsa-lib.git flameeyes
> 
> Diego E. 'Flameeyes' Pettenò (5):
>       Don't use AC_CANONICAL_SYSTEM, only use AC_CANONICAL_HOST.
>       Add the attributes.m4 macro file from xine/lscube.
>       Check for --no-undefined linker flag and use it.
>       Make sure that python libraries are passed through LIBADD.
>       Make some static tables and strings constants.

These changes look OK, but could you add your sign-off to each commit?

> git://git.flameeyes.eu/others/alsa/alsa-plugins.git flameeyes
> 
> Diego E. 'Flameeyes' Pettenò (6):
>       Add the attributes.m4 macro file from xine/lscube.
>       Check for --no-undefined linker flag and use it.
>       Cleanup flags in maemo/Makefile.am
>       [RFC] Don't use pow() for calculating a power of 2, use shift instead.
>       Make some static tables and strings constants.
>       Mark as static the functions not used outside their unit.

Ditto.

> git://git.flameeyes.eu/others/alsa/alsa-utils.git flameeyes
> 
> Diego E. 'Flameeyes' Pettenò (4):
>       Remove some dead code (comparisons between 0 and unsigned integers).

The change in alsactl/state.c gives wrong indentations.
Please fix.

>       Remove some unused variables.
>       Mark static the functions not used outside their unit.
>       Make some static tables and strings constants.

Give your sign-off, please, too.


thanks,

Takashi

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

* Re: Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils
  2008-11-21 11:52 ` Takashi Iwai
@ 2008-11-21 12:02   ` Diego E. 'Flameeyes' Pettenò
  2008-11-21 12:56     ` Takashi Iwai
  2008-11-21 12:12   ` Diego E. 'Flameeyes' Pettenò
  1 sibling, 1 reply; 6+ messages in thread
From: Diego E. 'Flameeyes' Pettenò @ 2008-11-21 12:02 UTC (permalink / raw)
  To: alsa-devel


[-- Attachment #1.1: Type: text/plain, Size: 311 bytes --]

Takashi Iwai <tiwai@suse.de> writes:

> These changes look OK, but could you add your sign-off to each commit?

Sorry, I forgot about it :/ Do you know if there is a setting in
git-config to get it enabled on a per-repository level by chance?

-- 
Diego "Flameeyes" Pettenò
http://blog.flameeyes.eu/

[-- Attachment #1.2: Type: application/pgp-signature, Size: 196 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

* Re: Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils
  2008-11-21 11:52 ` Takashi Iwai
  2008-11-21 12:02   ` Diego E. 'Flameeyes' Pettenò
@ 2008-11-21 12:12   ` Diego E. 'Flameeyes' Pettenò
  2008-11-21 13:00     ` Takashi Iwai
  1 sibling, 1 reply; 6+ messages in thread
From: Diego E. 'Flameeyes' Pettenò @ 2008-11-21 12:12 UTC (permalink / raw)
  To: alsa-devel


[-- Attachment #1.1: Type: text/plain, Size: 294 bytes --]

Takashi Iwai <tiwai@suse.de> writes:

> These changes look OK, but could you add your sign-off to each commit?

All commits have sign-off now and the indentation problem is fixed, and
pushed (forced update/history rewritten).

-- 
Diego "Flameeyes" Pettenò
http://blog.flameeyes.eu/

[-- Attachment #1.2: Type: application/pgp-signature, Size: 196 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

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

* Re: Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils
  2008-11-21 12:02   ` Diego E. 'Flameeyes' Pettenò
@ 2008-11-21 12:56     ` Takashi Iwai
  0 siblings, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2008-11-21 12:56 UTC (permalink / raw)
  To: Diego E. 'Flameeyes' Pettenò; +Cc: alsa-devel

At Fri, 21 Nov 2008 13:02:39 +0100,
Diego E. 'Flameeyes' Pettenò wrote:
> 
> Takashi Iwai <tiwai@suse.de> writes:
> 
> > These changes look OK, but could you add your sign-off to each commit?
> 
> Sorry, I forgot about it :/ Do you know if there is a setting in
> git-config to get it enabled on a per-repository level by chance?

No standard config I know of.  But it shouldn't be hard.


Takashi

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

* Re: Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils
  2008-11-21 12:12   ` Diego E. 'Flameeyes' Pettenò
@ 2008-11-21 13:00     ` Takashi Iwai
  0 siblings, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2008-11-21 13:00 UTC (permalink / raw)
  To: Diego E. 'Flameeyes' Pettenò; +Cc: alsa-devel

At Fri, 21 Nov 2008 13:12:14 +0100,
Diego E. 'Flameeyes' Pettenò wrote:
> 
> Takashi Iwai <tiwai@suse.de> writes:
> 
> > These changes look OK, but could you add your sign-off to each commit?
> 
> All commits have sign-off now and the indentation problem is fixed, and
> pushed (forced update/history rewritten).

Thanks, pulled in, pushed out now.


Takashi

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

end of thread, other threads:[~2008-11-21 13:00 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-21  2:50 Fixes and misc improvements for alsa-lib, alsa-plugins, alsa-utils Diego E. 'Flameeyes' Pettenò
2008-11-21 11:52 ` Takashi Iwai
2008-11-21 12:02   ` Diego E. 'Flameeyes' Pettenò
2008-11-21 12:56     ` Takashi Iwai
2008-11-21 12:12   ` Diego E. 'Flameeyes' Pettenò
2008-11-21 13:00     ` Takashi Iwai

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.