<table><tr><td style="">cordlandwehr marked an inline comment as done.<br />cordlandwehr added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27741">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D27741#inline-160190">View Inline</a><span style="color: #4b4d51; font-weight: bold;">cgiboudeaux</span> wrote in <span style="color: #4b4d51; font-weight: bold;">Big5Freq.tab:3</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">You added "Netscape Communications Corporation" on other files but not this one. (IMO, you shouldn't, Netscape is explicitly mentioned only in src/probers/nsSJISProber.h)</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I checked in commit <a href="https://phabricator.kde.org/R446:d761a5c62336b6d35e961a3388e65e187bea69c3" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">d761a5c62336b6d35e961a3388e65e187bea69c3</a> in the kdelibs repository and for the other files recovered the original copyright holder names that unfortunately were lost during the license change. yet for this file, even in the original file no copyright holder is named explicitly, only copyright for "Initial Developer" is named without specifying the initial developer. For the other files the "Initial Developer" explicitly is set to "Netscape Communications Corporation".</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R270 KCodecs</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27741">https://phabricator.kde.org/D27741</a></div></div><br /><div><strong>To: </strong>cordlandwehr<br /><strong>Cc: </strong>cgiboudeaux, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns<br /></div>