[QGHG-it-dev-list] SimpComp klasa

[Marko Vojinovic]

Evo uspeo sam malo da "pospremim" SimpComp klasu --- dodao sam gomilu komentara, reorganizovao redosled f-ja i grupisao ih, ubacio odredjen broj log_report() f-ja koje treba na raznoraznim mestima u kodu da hvataju i prijavljuju bilo sta neobicno (nullptr-e, prazne skupove, iskakanje iz domena za dimenziju i level, itdisl...). Plan je da u nastavku isto tako pocistim i doteram i ostale klase, jednu po jednu (redom prvo KSimplex klasu, pa Color, pa ostalo...).

Usput sam naleteo i na nekoliko manjih greskica (koje se mahom nisu ispoljavale u kodu), i jedan veci bag (f-ja reconstruct_neighbors_from_vertices() nije radila svoj posao kako treba...), i to sam popravio.

U fajlovima simpcomp.hpp i simpcomp.cpp sam ostavio i nekoliko internih komentara (na srpskom), koji su obelezeni sa "TODO" i "FIXME", koji se odnose na stvari koje nisam pokusao da popravljam, a verovatno bi trebalo. Bacite pogled na te komentare, pa se javite da prodiskutujemo sta bi bilo najpametnije uraditi za svaki od njih.

Inace, bilo bi dobro sledece nedelje da se cujemo online (bar Dusan, Nenad i ja, ostali kako zele), pa da nam Dusan malo usmeno pomogne da razumemo kako radi cmake. Ovo sto je do sada realizovano u cmake_setup branch-u izgleda dobro i radi dobro, ali bih voleo da bolje razumem celu filozofiju oko cmake i na koje sve nacine je moguce da se organizuje, pa onda da smislimo kako je najpametnije da namestimo cmake za nas projekt, i da onda ujedinimo cmake-branch u main-branch. Javicu se tokom vikenda sa predlogom za termin za sastanak sledece nedelje.

Od Nenada treba da stigne resenje zadatka 33, i vezano za to bacite pogled na "FIXME" komentar koji sam stavio u KSimplex.cpp, vezano za f-ju KSimplex::delete_all_neighbors(), gde mi se cini da mozda ima potencijalni bag/memory-leak... To nisam jos stigao da testiram i eventualno popravim, mada ne bi trebalo da bude mnogo tesko (umesto for-petlje stavimo while-petlju). Bacite pogled i na to, pa mi javite komentare.

:-)
Marko


Dr. Marko Vojinovic
Group for Gravitation, Particles and Fields
Institute of Physics
University of Belgrade
======================
home page: www.markovojinovic.com
e-mail:    vmarko at ipb.ac.rs