processing / processing/processing4
noStroke makes PShape.setTexture crash (P2D)
Personne n'a encore pris cette issue.
- Langage dominant
- Java
- Étoiles
- 494
- Forks
- 183
- Merge moyen
- 4 h 39 min
- PR mergées (30 j)
- 3
Description
Created by: TidensBarn
I had inexplicable crashes when using textures on PShapes that almost made me loose my will to live until I finally found the cause:
If I create a PShape in "noStroke" state, setTexture will work the first time, but when I then try to set a different texture and redraw, it'll crash:
PShape shape;
void settings() {
size(400, 400, P2D);
}
void setup() {
}
void draw() {
noStroke(); // Remove this, no crash
shape = createShape(RECT, 0, 0, 200, 200);
PImage p1 = loadImage("img1.png");
shape.setTexture(p1);
shape(shape);
PImage p2 = loadImage("img2.png"); // p2 needs to be a different size from p1
shape.setTexture(p2);
shape(shape); // Crash!
}
It's an ArrayIndexOutofBoundsException in PShapeOpenGL, line 843, because the field "firstLineVertex" happens to be -1...
Here's a sketch with images: crash.zip
Tested on Win 10, Processing 4.2
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par le sketch de plantage fourni et inspectez PShapeOpenGL autour de la ligne 843, où firstLineVertex est signalé comme valant -1. Reproduisez le problème dans P2D avec noStroke(), puis remplacez la texture d'une forme par une image d'une taille différente et redessinez-la. C'est terminé lorsque cette séquence ne provoque plus de plantage.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- java
- Domaine
- computer-graphics
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100