OpenJDK : un rapport de bogue souligne la présence d'expressions grossières au sein du code source
De l'implémentation open source de Java SE

Le , par Patrick Ruiz

61PARTAGES

13  0 
Les revues de code, notamment, de la section réservée aux commentaires peuvent s’avérer très intéressantes quant aux possibles trouvailles. Un contributeur au projet OpenJDK s’est prêté au jeu avec le code source de l’implémentation open source de Java SE et il faut dire que les développeurs qui sont passés par là n’ont pas « lésiné sur les mots. »

Illustration avec la ligne 1475 de cette portion de code au sein de la bibliothèque softchannel.java :

Code java : Sélectionner tout
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
1455            controlChange(100, 127); // RPN = Null
1456             controlChange(101, 127); // RPN = Null
1457 
1458             // see DLS 2.1 (Power-on Default Values)
1459             if (allControls) {
1460 
1461                 keybasedcontroller_active = null;
1462                 keybasedcontroller_value = null;
1463 
1464                 controlChange(7, 100); // Volume
1465                 controlChange(10, 64); // Pan
1466                 controlChange(91, 40); // Reverb
1467 
1468                 for (int controller : co_midi_rpn_rpn.keySet()) {
1469                     // don't reset tuning settings
1470                     if (controller != 3 && controller != 4)
1471                         rpnChange(controller, 0);
1472                 }
1473                 for (int controller : co_midi_nrpn_nrpn.keySet())
1474                     nrpnChange(controller, 0);
1475                 rpnChange(0, 2 << 7);   // Bitch Bend sensitivity
1476                 rpnChange(1, 64 << 7);  // Channel fine tunning
1477                 rpnChange(2, 64 << 7);  // Channel Coarse Tuning
1478                 rpnChange(5, 64);       // Modulation Depth, +/- 50 cent
1479 
1480                 tuning_bank = 0;
1481                 tuning_program = 0;
1482                 tuning = new SoftTuning();
1483 
1484             }
1485 
1486         }
1487     }
1488 
1489     @Override
1490     public void allNotesOff() {
1491         if (current_mixer != null)
1492             current_mixer.allNotesOff();
1493         synchronized (control_mutex) {
1494             for (int i = 0; i < voices.length; i++)
1495                 if (voices[i].on && voices[i].channel == channel

Le commentaire qui accompagne la ligne 136 au sein du code source de la bibliothèque java de manipulation de bits (BitArray.java) est tout aussi digne d’intérêt.

Code java : Sélectionner tout
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
 116      */
 117     public final int getNextBit(int startBit) {
 118         for (int i = (startBit >>> 5) ; i<=_intSize; i++) {
 119             int bits = _bits[i];
 120             if (bits != 0) {
 121                 for (int b = (startBit % 32); b<32; b++) {
 122                     if ((bits & _masks[b]) != 0) {
 123                         return((i << 5) + b);
 124                     }
 125                 }
 126             }
 127             startBit = 0;
 128         }
 129         return(DTMAxisIterator.END);
 130     }
 131 
 132     /**
 133      * This method returns the Nth bit that is set in the bit array. The
 134      * current position is cached in the following 4 variables and will
 135      * help speed up a sequence of next() call in an index iterator. This
 136      * method is a mess, but it is fast and it works, so don't fuck with it.
 137      */
 138     private int _pos = Integer.MAX_VALUE;
 139     private int _node = 0;
 140     private int _int = 0;
 141     private int _bit = 0;
 142 
 143     public final int getBitNumber(int pos) {
 144 
 145         // Return last node if position we're looking for is the same
 146         if (pos == _pos) return(_node);
 147 
 148         // Start from beginning of position we're looking for is before
 149         // the point where we left off the last time.
 150         if (pos < _pos) {
 151             _int = _bit = _pos = 0;
 152         }
 153 
 154         // Scan through the bit array - skip integers that have no bits set
 155         for ( ; _int <= _intSize; _int++) {
 156             int bits = _bits[_int];

Bitc*, f*ck, etc. D’après Adam Farley, on dénombre au minimum 12 expressions de ce type au sein de la bibliothèque open source et on peut allonger la liste si on fouille dans les dictionnaires dédiés aux phases de test de logiciels. « Le code source d’OpenJDK intègre trop de jurons disséminés un peu partout. Étant donné qu’OpenJDK est utilisé dans un contexte professionnel, il semble inapproprié de laisser ces 12 instances dans le code », écrit-il. Adam Farley a proposé une version préliminaire du jeu de modifications qu’il entend introduire. On verra par exemple un « Bitc* bend sensitivity » céder sa place à « Pitch bend sensitivity. »

Code java : Sélectionner tout
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
1455             controlChange(100, 127); // RPN = Null
1456             controlChange(101, 127); // RPN = Null
1457 
1458             // see DLS 2.1 (Power-on Default Values)
1459             if (allControls) {
1460 
1461                 keybasedcontroller_active = null;
1462                 keybasedcontroller_value = null;
1463 
1464                 controlChange(7, 100); // Volume
1465                 controlChange(10, 64); // Pan
1466                 controlChange(91, 40); // Reverb
1467 
1468                 for (int controller : co_midi_rpn_rpn.keySet()) {
1469                     // don't reset tuning settings
1470                     if (controller != 3 && controller != 4)
1471                         rpnChange(controller, 0);
1472                 }
1473                 for (int controller : co_midi_nrpn_nrpn.keySet())
1474                     nrpnChange(controller, 0);
1475                 rpnChange(0, 2 << 7);   // Pitch Bend sensitivity
1476                 rpnChange(1, 64 << 7);  // Channel fine tunning
1477                 rpnChange(2, 64 << 7);  // Channel Coarse Tuning
1478                 rpnChange(5, 64);       // Modulation Depth, +/- 50 cent
1479 
1480                 tuning_bank = 0;
1481                 tuning_program = 0;
1482                 tuning = new SoftTuning();
1483 
1484             }
1485 
1486         }
1487     }
1488 
1489     @Override
1490     public void allNotesOff() {
1491         if (current_mixer != null)
1492             current_mixer.allNotesOff();
1493         synchronized (control_mutex) {
1494             for (int i = 0; i < voices.length; i++)
1495                 if (voices[i].on && voices[i].channel == channel

Ou encore un « method is as mess, but it is fast and it works, so don’t fuck with it » faire ses adieux a profit de « method is as mess, but it is fast and it works, so don’t change it. »

Code java : Sélectionner tout
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 
 116      */
 117     public final int getNextBit(int startBit) {
 118         for (int i = (startBit >>> 5) ; i<=_intSize; i++) {
 119             int bits = _bits[i];
 120             if (bits != 0) {
 121                 for (int b = (startBit % 32); b<32; b++) {
 122                     if ((bits & _masks[b]) != 0) {
 123                         return((i << 5) + b);
 124                     }
 125                 }
 126             }
 127             startBit = 0;
 128         }
 129         return(DTMAxisIterator.END);
 130     }
 131 
 132     /**
 133      * This method returns the Nth bit that is set in the bit array. The
 134      * current position is cached in the following 4 variables and will
 135      * help speed up a sequence of next() call in an index iterator. This
 136      * method is a mess, but it is fast and it works, so don't change it.
 137      */
 138     private int _pos = Integer.MAX_VALUE;
 139     private int _node = 0;
 140     private int _int = 0;
 141     private int _bit = 0;
 142 
 143     public final int getBitNumber(int pos) {
 144 
 145         // Return last node if position we're looking for is the same
 146         if (pos == _pos) return(_node);
 147 
 148         // Start from beginning of position we're looking for is before
 149         // the point where we left off the last time.
 150         if (pos < _pos) {
 151             _int = _bit = _pos = 0;
 152         }
 153 
 154         // Scan through the bit array - skip integers that have no bits set
 155         for ( ; _int <= _intSize; _int++) {
 156             int bits = _bits[_int];

D’après ce qui ressort du rapport de bogue, la communauté n'est pas en total accord avec la proposition. Une version étendue des modifications est à paraître et prendra en compte les consensus, notamment, sur les termes à considérer comme des jurons. On parle du pouvoir des mots et un observateur qui s’inscrit en faux avec l’initiative commente qu’ « un développeur est libre de faire usage d’un juron si celui-ci constitue le meilleur moyen de faire passer un message. »


Il y a quelques mois, la communauté Python s’est engagée sur un front similaire. En septembre, elle a engagé le processus de suppression des termes « master » et « slave » de sa documentation et de sa base de code. On évoquait alors la nécessité de prendre de la distance d’avec ces derniers pour des raisons de diversité et pour leur connotation à l’esclavage.

Source : rapport de bogue

Et vous ?

Que pensez-vous de l’initiative ?

Que pensez-vous de l’utilisation des jurons dans l’univers du génie logiciel ?

Vous est-il arrivé d’en faire usage ? Si oui, quel était l’objectif à atteindre ?

Êtes vous d’accord avec l’observateur qui affirme qu’ « un développeur est libre de faire usage d’un juron si celui-ci constitue le meilleur moyen de faire passer un message » ?

Voir aussi :

Python va supprimer les termes "master/slave" de sa documentation et sa base de code pour des raisons de diversité et leur connotation à l'esclavage

Andrew Haley de Red Hat donne son avis sur l'avenir d'OpenJDK, l'implémentation libre de Java SE, après la sortie de JDK 11 sous licence commerciale

OpenJDK pour MacOS : le projet a débuté, les premiers codes sont disponibles

Java SE : Oracle soumet l'utilisation des fonctionnalités commerciales à une souscription mensuelle, la licence permanente n'est plus d'actualité

OpenJDK bientôt présent sur Windows Azure, Microsoft travaille sur une implémentation open source de « Java as a Service »

Une erreur dans cette actualité ? Signalez-le nous !

Avatar de Darktib
Membre confirmé https://www.developpez.com
Le 16/12/2018 à 23:06
Quel puritanisme...
Avatar de Neckara
Expert éminent sénior https://www.developpez.com
Le 17/12/2018 à 0:18
12 jurons sur un code de combien de centaines de millier de lignes ?
D'autant que bon… on ne peut pas dire que ces jurons soient si choquant que cela.

Quitte à parler de "contexte professionnel", perdre du temps sur 12 jurons dans un code aussi gros… bonjour le professionnalisme.
On croirait entendre des gamins de maternelles "han ! Ils ont écrit un groooooos mot !".

Cela m'a toujours amusé de voir les anglophones dire "f-word". Ici ils vont jusqu'à dire "f-bombs ".
Ce serait des enfants qui ont peur de se prendre une torgnole de leur parents s'ils jurent, je comprendrais. Mais là on parle d'adultes… qui ont peur de dire "fuck"…

La communauté n'est pas totalement d'accord avec la correction.

Sweet! Learning English with the OpenJDK community.

You sir, probably, have a degree in public relations and/or marketing. This was*the best* way to draw attention to those swear words. Had these words stayedunexposed, no one would have been bothered by them. I guess...

On a serious note, I wouldn't change cave art just because we might find it inappropriate, it's history now.
http://mail.openjdk.java.net/piperma...er/057308.html

What is this, the thought police?
http://mail.openjdk.java.net/piperma...er/057338.html

It also would be good to discuss the instances that you’ve proposedto change in your patch.

I can certainly see removing the f-word, and other words of a sexualnature. Those are clearly inappropriate.

Removing lesser words, and continuing to police their use henceforth, strikes me as overkill.

What do other Committers think?
http://mail.openjdk.java.net/piperma...er/057345.html

I don't think our sensibilities need be so delicate as to be compelled to purge mild exclamations from test files.

As noted elsewhere on this thread, the words altered in the proposed patch which intersect with the words in George Carin's list of "Seven Words You Can Never Say on Television," originate in up-stream sourcesand would be better addressed there.
http://mail.openjdk.java.net/piperma...er/057352.html

Zounds! Heavens to Betsy! Please defer this patch until April 1 2019.
http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-December/057369.html


Sur reddit aussi la modification ne fait pas l'unanimité.
Fucking snowflakes.
Fuck raw string literals. I just want to get rid of those darn swear words.
Fuck you!
I want my god dam string literals
„Mom! Get the Camera, i‘m improving open source!“
You can never have too many swear words.
Even though I agree when it comes to swear words in code, but I wouldn't support this request. Well, removing swears words in comments can be done at any time while refactoring, but renaming the whole package because it's "crap" is pathetic.

Sorry dude who opened that bug, but you should get some work or strengthen your focus..
Well, that’s a lack of focus.
What the fork?!
What a stupid fucking patch. A god damn waste of time.
What a tremendous waste of time.
Fuck this shit! People should be allowed to express themselves. What is this thought-police bullshit?!
OMG. He went full retard here, and fixed somewhat external dependencies, like JQuery, making an update of JavaDoc tooling harder.
Please someone tell me this patch was rejected?

First word changes over being politically correct, now word changes over curses themselves? Hello 1984 here we come.
Wait, OpenJDK is not halal? get a life
I think the Java language is becoming too sensitive ...

Donc comme d'habitude on nous prend pour des cons.
Avatar de 23JFK
Membre expérimenté https://www.developpez.com
Le 17/12/2018 à 1:36
Qu'est-ce que les "bien-pensants" peuvent nous casser les couilles.
Avatar de gchauvet
Membre à l'essai https://www.developpez.com
Le 17/12/2018 à 10:21
Comme on dit par chez nous : "ça marche mieux avec une bonne pile de jurons"
Avatar de Steinvikel
Membre éprouvé https://www.developpez.com
Le 17/12/2018 à 12:10
Êtes vous d’accord avec l’observateur qui affirme qu’ « un développeur est libre de faire usage d’un juron si celui-ci constitue le meilleur moyen de faire passer un message » ?
Citation Envoyé par Adam Farley
Le code source d’OpenJDK intègre trop de jurons disséminés un peu partout. (...) dans un contexte professionnel, il semble inapproprié de laisser ces 12 instances dans le code
Citation Envoyé par Adam Farley
(...) prendra en compte les consensus, notamment, sur les termes à considérer comme des jurons. On parle du pouvoir des mots et un observateur (...)
J'entends tout à fait qu'il soit fatiguant de lire du code ou il est régulièrement exprimé de l'agressivité, vulgarité gratuite... d'autant plus en milieu professionnel.
Cependant, il est parfois préférable, voir nécessaire, de marquer les esprits, de ce faire entendre (lu, et pas survolé). La grande question sera de trancher pour déterminer la limite entre le défoulement et l'utilité (légitime ou pas d'ailleurs).
Quand à la liberté de faire usage de juron pour faire passer un message, le problème est que certains y voient une nécessité là où d'autres y vois du superflu inapproprié. Et ce phénomène dépend de l'éducation, de l'enseignement, des habitudes, des accoutumances, du conditionnement.

Que pensez-vous de l’utilisation des jurons dans l’univers du génie logiciel ?
l'humour, la défiance, la violence, le militantisme... ont leur place dans le code si ils servent une chose spécifique --> l'information, la compréhension, la sensibilisation, la prévention ...dans le domaine concerné, que ce soit directement, ou indirectement (pas trop indirect non plus, hein).

Citation Envoyé par Patrick Ruiz Voir le message
Il y a quelques mois, la communauté Python s’est engagée sur un front similaire. En septembre, elle a engagé le processus de suppression des termes « master » et « slave » de sa documentation et de sa base de code. On évoquait alors la nécessité de prendre de la distance d’avec ces derniers pour des raisons de diversité et pour leur connotation à l’esclavage.
"What the fork !?" (c'est si joliment dit)
master/slave est, en informatique, un concept opératoire, et non une idéologie politique... vont-ils refaire le monde parce que certains l'ont malmené ? --> interdiction de l'usage de palettes de couleurs faisant apparaitre un arc-en-ciel parce que ça pourrait déplaire aux gays, interdiction de l'emploi de "we can" parce que ce serait politiser du code... des conneries on en as sans fin en réserves.
NB: ils font comment ceux qui vont documenter du python sur du code orienté pour du matos MOSI/MISO (Master Output - Slave Input...)

Il n'y a que moi que ça choque ?
Il manquerait plus que les ethnologues aient l'interdiction d'employer le mot "race" dans leur métier juste parce que un gus à détourné ce mot pour faire le mal... un mot qui existait bien avant lui en classification taxonomique (un mot utile pour ranger les choses, pas pour les qualifier).

Que pensez-vous de l’initiative ?
très clivante, contextuelle, demande une réponse contextuelle... affaire à suivre.

Vous est-il arrivé d’en faire usage ? Si oui, quel était l’objectif à atteindre ?
oui, pour des commentaires d'alerte, sans passer dans le lexique fleuri, passer certains commentaires dans le registre de la vulgarité permet un effet d'emphase moins agressif visuellement que d'écrire en majuscules ou de multiplier les points d'exclamations.
NB: il ne faut pas tomber d'en l'emploi systématique de ce type de langage, c'est bien plus efficace quand il est utilisé seulement pour démarquer, pour explicité l'importance plus grande que l'on porte à un propos.
Avatar de professeur shadoko
Membre expérimenté https://www.developpez.com
Le 17/12/2018 à 13:14
il y a des nuances: les injures et l'agressivité ne sont pas de mise ... par contre une petite blague grivoise dans le nom d'une variable ou une contrepèterie bien menée dans le code ... ça agrémente le boulot!
(j'ai fait beaucoup de ça dans des scripts-shell! le soir on avait une petite compétition avec un professeur de médecine qui nous montrait ses transparents de cours avec ce genre de blague )
Avatar de OButterlin
Modérateur https://www.developpez.com
Le 03/04/2019 à 15:34
Damned, ce n'est même pas un poisson d'avril...

Master / Slave... ben oui, c'est bien de processus maîtres et d'autres esclaves dont on parle, où est le problème ? Il n'y a aucun rapport avec l'esclavage humain là
Dans le même registre, on ne pourra plus dire "texte blanc sur fond noir" ? ça fait raciste ? On sent la position dominante du blanc sur le noir ?
Quelle connerie

Pour le reste, j'avoue, j'ai déjà utilisé des commentaires légèrement agressifs dans des programmes... mais ça restait gentil.
D'abord ça m'a défoulé sur le coup, et en plus ça a fait rire les autres développeurs qui l'ont lu, alors, pourquoi s'en priver
Contacter le responsable de la rubrique Accueil

Partenaire : Hébergement Web