poke-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] crc.pk: New file


From: Jose E. Marchesi
Subject: Re: [PATCH] crc.pk: New file
Date: Wed, 27 Nov 2019 16:10:55 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

         
         If this is a pickle, it should go to pickle/ instead of src/.
    
    Why make the distinction?  After all, the all end up getting installed
    in the same place.

src/ is the place for rt.pk (the compiler runtime) and std.pk (the
standard library).

At some point we may want to move std.pk into a hierarchy of files under
pickles/ but not now.

         Is this crc32_update function useful by itself?  If not, I would have 
it
         defined inside crc32_cal, called `update'.
    
    I can envisage a scenarios where it might be useful by itself, although
    admittedly such situations would be rare.
         
    J'



reply via email to

[Prev in Thread] Current Thread [Next in Thread]