[chora] patch for htmlspaces

Cynic cynic@mail.cz
Wed, 23 May 2001 19:43:19 +0200


Ok, but then the description should be changed to reflect that.
That's what it says now:

/**
   * Convert a line of text to be HTML-displayable
   * @param text The line of text to convert
   * @return The HTML-compliant converted text.  It always returns at least
                                                    ^^^^^^^^^^^^^^^^^^^^^^^
   *         a non-breakable space, if the return would otherwise be empty.
             ^^^^^^^^^^^^^^^^^^^^^
   */


At 13:32 23.5. 2001 -0400, you wrote:
>Quoting Cynic <cynic@mail.cz>:
>
> > Whatever... your patch is wrong - if you call htmlspaces() with
> > an empty string, it returns it untouched. That's not what its
> > description says. This is a correct patch:
>
>I think returning it untouched is correct behavior (especially in a library
>function, now that it's moved to Text::), so I've kept that. If that causes
>problems, they should be resolved elsewhere, not in the library function.
>
>-chuck
>
>--
>Charles Hagenbuch, <chuck@horde.org>
>Black and white and grey, all the shades of truth.
>
>--
>Chora mailing list: http://horde.org/chora/
>Frequently Asked Questions: http://horde.org/faq/
>To unsubscribe, mail: chora-unsubscribe@lists.horde.org



cynic@mail.cz
-------------
And the eyes of them both were opened and they saw that their files
were world readable and writable, so they chmoded 600 their files.
     - Book of Installation chapt 3 sec 7