mtd-utils: fec.c uses bcopy()

Mitch Davis mjd-linux-mtd at afork.com
Tue Oct 7 22:36:13 EDT 2008


On Wed, Oct 8, 2008 at 12:02 PM, Mike Frysinger <vapier.adi at gmail.com> wrote:
> On Tue, Oct 7, 2008 at 20:27, Mitch Davis wrote:
>
>> fec.c in mtd-utils uses bcopy(), so mtd-utils doesn't compile for me.
>> I'd like to fix this.
>>
>>  - Should I rewrite fec.c so it uses memcpy instead?
>>  - Should I just add #define NEED_BCOPY to fec.c?
>>  - Should I leave fec.c unchanged and add -DNEED_BCOPY to the Makefile
>> (which would be passed to every compile?)
>
> a simple sed can fix the code as the only difference between bcopy()
> and memcpy() is the argument order.  ifdef's are pointless/ugly in
> this case.

I agree.  I've attached a diff that makes this change.

It compiles with no warnings, but I have no way of testing it.  Any
suggestions what to do next?

Thanks,

Mitch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fec.diff
Type: application/octet-stream
Size: 3974 bytes
Desc: not available
Url : http://lists.infradead.org/pipermail/linux-mtd/attachments/20081008/fc9d0041/attachment.obj 


More information about the linux-mtd mailing list