[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bugs-dist
Subject:    [Bug 109836] Add Support for displaying Face: header
From:       Nick Brown <nickbroon () blueyonder ! co ! uk>
Date:       2005-08-21 22:58:02
Message-ID: 20050821225802.29029.qmail () ktown ! kde ! org
[Download RAW message or body]

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
         
http://bugs.kde.org/show_bug.cgi?id=109836         
nickbroon blueyonder co uk changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #12233|0                           |1
        is obsolete|                            |
  Attachment #12235|0                           |1
        is obsolete|                            |



------- Additional Comments From nickbroon blueyonder co uk  2005-08-22 00:58 -------
Created an attachment (id=12315)
 --> (http://bugs.kde.org/attachment.cgi?id=12315&action=view)
updated patch

I've made the suggest changes.
The use of TRUE has been removed.
The header length check comment updated, and length of the header name taken
into account.
The code updated to meet the coding style guidelines, except for line length
but this seems to be widely ignored for debug lines. Please check for further
violations. (It should be noted I had simply been following the X-Face code in
the same file for previous patches.)
[prev in list] [next in list] [prev in thread] [next in thread]