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

Re: [PATCH] gitweb: use decode_utf8 directly

From
IDIsmail Dönmez <ismail@pardus.org.tr>
Date
Jun 1, 2007, 19:47 UTC
Message-ID
<200706012247.57273.ismail@pardus.org.tr>
In-Reply-To
<7vr6ovzcgr.fsf@assigned-by-dhcp.cox.net>
On Friday 01 June 2007 22:44:36 Junio C Hamano wrote:
Show 9 quoted lines
> Alexandre Julliard <julliard@winehq.org> writes:
> > Sorry for the late nack, but it turns out that this patch breaks diff
> > output on the Wine server for files that are not utf-8.
> >
> > The cause is apparently that decode_utf8() returns undef for invalid
> > sequences instead of substituting a replacement char like
> > decode("utf8") does.
>
> Thanks for noticing.  Will revert.

Why are reverting a correct bugfix? :( He's at most using outdated software. *sigh*

/ismail
-- 
Perfect is the enemy of good
Previous: Junio C HamanoNext: Junio C Hamano
Message 16 of 21 in “gitweb: use decode_utf8 directly”
  1. gitweb: use decode_utf8 directlyIsmail Dönmez, Apr 24, 2007
  2. Ismail DönmezApr 27, 2007
  3. Junio C HamanoApr 27, 2007
  4. Ismail DönmezApr 27, 2007
  5. Junio C HamanoApr 27, 2007
  6. Ismail DönmezMay 1, 2007
  7. Junio C HamanoMay 1, 2007
  8. Ismail DönmezMay 1, 2007
  9. Ismail DönmezMay 1, 2007
  10. Ismail DönmezMay 3, 2007
  11. Junio C HamanoMay 3, 2007
  12. Alexandre JulliardJun 1, 2007
  13. Ismail DönmezJun 1, 2007
  14. Alexandre JulliardJun 1, 2007
  15. Junio C HamanoJun 1, 2007
  16. Ismail DönmezJun 1, 2007
  17. Junio C HamanoJun 1, 2007
  18. Ismail DönmezJun 1, 2007
  19. Junio C HamanoJun 3, 2007
  20. Ismail DönmezJun 3, 2007
  21. Jakub NarebskiJun 2, 2007

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.