bmeaut / grainautline

GrainAutLine: image processing for geology and material sciences
GNU General Public License v3.0
0 stars 2 forks source link

Ez alatt a kommentár alatt mire gondoltál? #109

Closed csorbakristof closed 8 years ago

csorbakristof commented 8 years ago

diff --git a/MarbleCommon/BlobsAndCurves/ProcessingStateDescriptor.h b/MarbleCommon/BlobsAndCurves/ProcessingStateDescriptor.h index a707511..55263b2 100644 --- a/MarbleCommon/BlobsAndCurves/ProcessingStateDescriptor.h +++ b/MarbleCommon/BlobsAndCurves/ProcessingStateDescriptor.h @@ -17,6 +17,9 @@ public: /* Loads from a file created by Save(). / explicit ProcessingStateDescriptor(std::string filename);

Mivel másolás történik az std::async hívásoknál, illetve fordítótól függően a SlotManager.cpp:145-ben, így érdemes explicite kiírni, hogy a másoló konstruktor az alapértelmezett, hogy még véletlenül se higgye azt senki, hogy a setId() ott is növel.

szotsaki commented 8 years ago

Mindössze egy ProcessingStateDescriptor(const ProcessingStateDescriptor& other) = default; deklarálást, hogy az egyszeri ember, aki a PSD kódja felé téved, lássa, hogy másolásnál szándékosan nincs ID növelgetés és nem csak lemaradt.

szotsaki commented 8 years ago

Fixed in 0d62bcf6ba7fa61e5078da8a1550071bb14547b8.