git/list[1] front-page[2] threads[3] people[4] search[5] about
 

base85: Two tiny fixes

From
AGAndreas Gruenbacher <agruen@suse.de>
Date
Jan 7, 2010, 14:58 UTC
Message-ID
<201001071558.30065.agruen@suse.de>

While looking at the base85 code I found a bug in the debug code and an unnecessary call. You may want to have a look at the two fixes here:

  http://www.kernel.org/pub/scm/linux/kernel/git/agruen/git.git

There is another little oddity in the way the de85 table is set up: 0 indicates an invalid entry; to avoid this from clashing with a valid entry, valid entries are incremented by one and decremented again while decoding. This leads to slightly worse code than using a negative number to indicate invalid values (and avoiding to increment/decrement).

Andreas
Next: Nicolas Pitre
Message 1 of 11 in “base85: Two tiny fixes”
  1. Andreas GruenbacherJan 7, 2010
  2. Nicolas PitreJan 7, 2010
  3. Andreas GruenbacherJan 8, 2010
  4. 1/3 base85 debug code: Fix length byte calculationAndreas Gruenbacher, Jan 8, 2010
  5. 2/3 base85: No need to initialize the decode table in encode_85Andreas Gruenbacher, Jan 8, 2010
  6. Michael J GruberJan 8, 2010
  7. Junio C HamanoJan 8, 2010
  8. base85: encode85() does not use the decode tableAndreas Gruenbacher, Jan 8, 2010
  9. base85: encode_85() does not use the decode tableAndreas Gruenbacher, Jan 8, 2010
  10. 3/3 base85: Make the code more obvious instead of explaining the non-obviousAndreas Gruenbacher, Jan 8, 2010
  11. A Large Angry SCMJan 8, 2010

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.