public inbox for ecos-discuss@sourceware.org
 help / color / mirror / Atom feed
From: "Steve Strublic" <SStrublic@hypercom.com>
To: andrew@lunn.ch
Cc: ecos-discuss@sources.redhat.com
Subject: Re: [ECOS] No zlib crc32 support?
Date: Mon, 15 Dec 2003 17:56:00 -0000	[thread overview]
Message-ID: <OF4986D169.3A0DE40A-ON07256DFD.0054E063-07256DFD.0055403E@hypercom.com> (raw)


I'll leave the resolution up to the maintainers.

Steve

--------
"Space is almost infinite.  In fact, we think it is infinite."  -- Dan Quayle



                                                                                                                                                       
                      andrew@lunn.ch                                                                                                                   
                                               To:       SStrublic@hypercom.com                                                                        
                      12/14/03 01:51 PM        cc:       ecos-discuss@sources.redhat.com                                                               
                                               Subject:  Re: [ECOS] No zlib crc32 support?                                                             
                                                                                                                                                       




On Fri, Dec 12, 2003 at 09:49:39AM -0700, Steve Strublic wrote:
>
> Andrew, that's why I chose zlib's crc32 algorithm.

So you have chosen the algorithm.  That algorithm happens to be
implemented by zlib as crc32 on unix like platforms. On eCos its
implemented by, according to you, cyg_ether_crc32_accumulate.

So we have no problems here.

> Andrew, you state... "The maintainers have no intention of changing
> these functions unless somebody can prove they are broken, which i
> think it very unlikely."

> I think I may have just proved that there's an issue.  I don't think
> it's proper to have to #define 'crc32' as'
> cyg_ether_crc32_accumulate'.  I'd prefer to hear the maintainers'
> position firsthand.

Why don't you think this is proper?

As far as im aware there is no standard covering this issue. The
proprietary library zlib is polluting the namespace by declaring this
function. This may cause problems in the future if POSIX do decide to
have a function called crc32() and decide to uses the POSIX crc
algorithm.

By using the names cyg_* i've deliberately avoided this namespace
pollution leaving the application and POSIX the ability to do what it
wants.

        Andrew





-- 
Before posting, please read the FAQ: http://sources.redhat.com/fom/ecos
and search the list archive: http://sources.redhat.com/ml/ecos-discuss

             reply	other threads:[~2003-12-15 17:36 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-12-15 17:56 Steve Strublic [this message]
2003-12-15 19:52 ` Nick Garnett
  -- strict thread matches above, loose matches on Subject: below --
2003-12-12 22:35 Steve Strublic
2003-12-15  8:33 ` Andrew Lunn
2003-12-10 16:22 Steve Strublic
2003-12-10 16:14 Steve Strublic
2003-12-10 16:20 ` Gary Thomas
2003-12-11  0:38 ` Andrew Lunn
2003-12-09 21:08 Steve Strublic
2003-12-09 21:27 ` Gary Thomas
2003-12-10  8:39 ` Andrew Lunn
2003-12-09 20:29 Steve Strublic
2003-12-09 20:45 ` Gary Thomas

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=OF4986D169.3A0DE40A-ON07256DFD.0054E063-07256DFD.0055403E@hypercom.com \
    --to=sstrublic@hypercom.com \
    --cc=andrew@lunn.ch \
    --cc=ecos-discuss@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).