ЙажаСценарий / Говнокод #19685 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
// if too many vertices, start a new mesh;
            if (vertices.length > 65000) {
                // finalize the old mesh	
                mesh.triangles = triangles;
                mesh.vertices = vertices;
                mesh.colors = colors;
                mesh.computeWireframe();
                mesh.computeNormals();

                if (mesh.vertices.length) {
                    meshes.push(mesh);
                }

не добавляем пустые меши, нафиг они нам

Запостил: makc3d makc3d, (Updated )

Комментарии (8) RSS

  • > vertices.length > 65000
    > mesh.vertices = vertices;
    все нормально, условие всегда выполнится.
    Ответить
    • может тогда лучше написать if (true) короче и читабельней
      Ответить
      • Видимо, раньше проверки vertices.length > 65000 не было, и код был уместен. Ну а в таком варианте достаточно просто убрать этот if что компилятор, вероятнее всего, и сделает.
        Ответить
        • >Видимо, раньше проверки vertices.length > 65000 не было, и код был уместен.
          Просто если проверки не было (ну или как я подумал что просто скопировали кусок из ветки ниже, ведь что-то наверное и с мешами до 65к вершин делают), то код и был веселым говном - взяли меш, посчитали нормали и кокойто wireframe, а потом хоп - а он оказывается пустой, ну мы и забиваем на него, пусть с ним gc париться.
          Ответить

Добавить комментарий

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?


    8