This is the mail archive of the guile@sourceware.cygnus.com mailing list for the Guile project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: base64.scm version 0.1


thanks for the comments!

ttn> i haven't run the code (my guile is somewhat trashed at the moment :-),
ttn> but i do have a comment on the source: could you use symbols instead of
ttn> numbers for `current-state'?  that would make it easier to understand,
ttn> depending on your choice of symbol names.  

sure...but do you have any suggestions for the symbol names?

the following come to mind:

  'left-octet, 'middle-octet, 'right-octet

  'first-octet, 'second-octet, 'third-octet, 'fourth-octet

not sure if they are any better though.

ttn> you might also be able to replace the `cond' w/ a `case'.

i will look into `case'.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]