Skip to Content.
Sympa Menu

cc-devel - Re: [cc-devel] liblicense: why ll_license_list_t?

cc-devel AT lists.ibiblio.org

Subject: Developer discussion for Creative Commons technology and tools

List archive

Chronological Thread  
  • From: Jason Kivlighn <jkivlighn AT gmail.com>
  • To: Peter Miller <millerp AT canb.auug.org.au>
  • Cc: creative commons developers <cc-devel AT lists.ibiblio.org>
  • Subject: Re: [cc-devel] liblicense: why ll_license_list_t?
  • Date: Thu, 06 Sep 2007 00:57:09 -0700

Sounds like my code :-)

The short answer is avoid extra loops required to handle the complexity
of not knowing how many licenses are going to be in each list ahead of
time. The next to last loop in ll_new_license_chooser should show why
this is.

So that's the "internal" and perhaps lazy reason. If we want
consistency we can either do extra processing in ll_new_license_chooser,
or convert the linked-list to a null-terminated array in
ll_get_licenses_from_flags (although now that I write this down, this
second method would be either inefficient or inconsistent anyway).

Cheers,
Jason

Peter Miller wrote:
> Hi LibLicense developers,
>
> I've been reading the ll_license_chooser code. Why does
> ll_license_list_t exist? Why doesn't the license chooser code use a
> char** list like the rest of the code?
>
>
> Regards
> Peter Miller <millerp AT canb.auug.org.au>
> /\/\* http://miller.emu.id.au/pmiller/
>
> PGP public key ID: 1024D/D0EDB64D
> fingerprint = AD0A C5DF C426 4F03 5D53 2BDB 18D8 A4E2 D0ED B64D
> See http://www.keyserver.net or any PGP keyserver for public key.
>
> Theory is when you know something, but it doesn't work. Practice is
> when something works, but you don't know why. Programmers combine
> theory and practice: Nothing works and they don't know why.
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> cc-devel mailing list
> cc-devel AT lists.ibiblio.org
> http://lists.ibiblio.org/mailman/listinfo/cc-devel
>





Archive powered by MHonArc 2.6.24.

Top of Page