Change buffer-substring to buffer-substring-no-properties?

I think I found how to fix ilisp to work with emacs 21.2, but I want
to make sure that I'm not going to break an old version of xemacs or
emacs with the change.

A news group article about the issue with emacs 21.2 is cached with
google as:

http://groups.google.com/groups?q=emacs+buffer-substring+font&hl=en&lr=&ie=UTF-8&selm=mksnbgobfn.fsf%40news.franz.com&rnum=1

I have tested changing all buffer-substring calls to
buffer-substring-no-properties calls (except in comint-v18.el). This
appears to work fine in brief testing with emacs 20 & 21.2 and xemacs
21.

Is anyone familiar with this issue? Is there any reason why I should
should not commit this change to the CVS tree?

Thanks!

-- 
       Kevin Rosenberg        |  .''`.  ** Debian GNU/Linux **
  http://b9.com/debian.html   | : :' :      The  universal
  GPG signed and encrypted    | `. `'      Operating System
     messages accepted.       |   `-    http://www.debian.org/


-------------------------------------------------------
This sf.net email is sponsored by: Influence the future 
of Java(TM) technology. Join the Java Community 
Process(SM) (JCP(SM)) program now. 
http://ads.sourceforge.net/cgi-bin/redirect.pl?sunm0003en