[gstreamer-bugs] [Bug 542510] [apexsink] Apple AirPort Express Wireless Sink

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=542510

  GStreamer | gst-plugins-bad | Ver: 0.10.15

GRemi changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |gremimail at gmail.com




------- Comment #3 from GRemi  2008-07-16 09:42 UTC -------

> - Could you attach it as a diff to gst-plugins-bad, including build system
> modifications?

I'll do the diff once the other points done .

> - Why exactly is the openssl needed? It might make sense to use GnuTLS or
> libgcrypt instead for license reasons

Sure. I'll work on the gcrypt use instead of openssl (random bytes, RSA public
key encryption).

> - No // comments, please use /* */ comments instead, there are still some
> compilers that don't like them in C code
> - Add your real name at the head of the source code files for the copyright.
> some nickname won't give you anything
> - You could use the GLib base64 implementation instead of your own
> - Where does the AES code come from? Also, doesn't openssl/libgcrypt provide
> this already anyway? :)
> - No declarations after statements, i.e. move all variable declarations at the
> start of a block (same reason as for the // comments)
> 
> Apart from that the code looks really good and clean, I like it :)
> 

Thank you very much for for your advise and your time.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=542510.