On Thursday 30 April 2009 01:03:58 Daniel Cheng wrote: > On Thu, Apr 30, 2009 at 8:02 AM, Matthew Toseland > <toad at amphibian.dyndns.org> wrote: > > On Thursday 23 April 2009 02:40:26 j16sdiz at freenetproject.org wrote: > >> Author: j16sdiz > >> Date: 2009-04-23 01:40:25 +0000 (Thu, 23 Apr 2009) > >> New Revision: 27218 > >> > >> Modified: > >> ? ?trunk/contrib/fec/src/com/onionnetworks/fec/Native16Code.java > >> ? ?trunk/contrib/fec/src/com/onionnetworks/fec/Native8Code.java > >> ? ?trunk/contrib/fec/src/csrc/com_onionnetworks_fec_Native16Code.h > >> ? ?trunk/contrib/fec/src/csrc/com_onionnetworks_fec_Native8Code.h > >> ? ?trunk/contrib/fec/src/csrc/fec16-jinterf.c > >> ? ?trunk/contrib/fec/src/csrc/fec8-jinterf.c > >> Log: > >> Use GetLongField() / SetLongField() > >> > > ... > >> Modified: trunk/contrib/fec/src/csrc/fec8-jinterf.c > >> =================================================================== > >> --- trunk/contrib/fec/src/csrc/fec8-jinterf.c 2009-04-23 01:40:07 UTC (rev > > 27217) > >> +++ trunk/contrib/fec/src/csrc/fec8-jinterf.c 2009-04-23 01:40:25 UTC (rev > > 27218) > > ... > >> > >> -/* > >> - * The data[] MUST be preshuffled before this call is made or it WILL NOT > >> - * WORK! ?It is very difficult to make Java aware that the pointers have > >> - * been shuffled in the encode() call, so we must pre-shuffle the data > >> - * so that encode doesn't move any pointers around. > >> - */ > > > > Why delete this comment? And the one in fec16-jinterf too? > > No specific reason. > I was syncing fec8-jinterf.c and fec16-jinterf.c so they look identical > in layout and comments.
Both of them had that comment. Would you mind reinstating it? -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 835 bytes Desc: This is a digitally signed message part. URL: <https://emu.freenetproject.org/pipermail/devl/attachments/20090430/f17cfec9/attachment.pgp>
