2009/9/7 Roland Levillain <roland(a)lrde.epita.fr>fr>:
Alexandre Duret-Lutz <adl(a)lrde.epita.fr>
writes:
>>
"FF" == Fabien Freling <fabien.freling(a)lrde.epita.fr> writes:
FF> URL:
https://svn.lrde.epita.fr/svn/oln/trunk/milena/sandbox
FF> ChangeLog:
FF> 2009-09-04 Fabien Freling <fabien.freling(a)lrde.epita.fr>
FF> Implement HQ2x upscaling algorithm.
FF> * fabien/mln/upsampling/hq2x.hh: Implement HQ2x algorithm.
FF> * fabien/mln/upsampling/hq3x.hh: Remove dead code.
FF> * fabien/tests/upsampling/Makefile: Add `hq2x' target.
FF> * fabien/tests/upsampling/hq2x.cc: New test file for HQ2x.
Je suis assez choqué. Ce code est une adaptation d'un code
de Maxim Stepin sous LGPL 2.1 : on reconnaît clairement la
logique du code original, même si les accès à l'image ont changé.
Cependant
1) le nom de l'auteur n'apparaît nul part
2) son copyright a été modifié en (c) LRDE
3) la license a été changée en GPL 2 + exception.
Le dépôt SVN de milena est public. Tu ne peux pas prendre le
code de quelqu'un et le redistribuer sous une autre license
comme ça !
J'appuie. Il faut faire très attention à ça.
Lorsque l'on reprend du code, il faut laisser le copyright header à
l'identique : ce n'est pas parce que tu modifies un fichier que tu en
deviens légalement l'auteur.
Et ça fait partie des avantages du libre : on peut panacher des bouts de
code à condition de respecter le droit d'auteur et les termes de la
licence.
J'étais justement en train de reprendre les headers.
Ce premier commit etait surtout histoire de sauvegarder le travail.
Et
indépendamment du problème de copyright/license, quand on
implémente un algo précis, c'est une bonne idée d'en donner les
références (article, page web, ...) dans l'entête.
Absolument ! Il faut penser aux autres (nous + les utilisateurs
externes du projet).
C'est prévu dans le prochain commit avec les références nécessaires.
J'aurais du le faire au début, mea culpa.
--
Fabien Freling
Email: fabien.freling(a)gmail.com
Website:
http://www.ffreling.com
Phone: +33 (0)6 07 08 24 84