Skip to content

Commit

Permalink
Improve comments and fix typos
Browse files Browse the repository at this point in the history
  • Loading branch information
marcboulle committed Dec 20, 2024
1 parent 5ab495f commit 110b87f
Show file tree
Hide file tree
Showing 11 changed files with 18 additions and 15 deletions.
2 changes: 1 addition & 1 deletion src/Learning/KWData/JSONFile.h
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ class JSONFile : public TextService
// potentielles entres les caracteres ansi initiaux recodes en utf8 avec windows1252/iso8859-1
// et les caracteres utf8 initiaux presents dans windows1252/iso8859-1.
// En cas de collision, les deux champs supplementaires sont ajoutes dans le json pour aide
// au diagnostique:
// au diagnostic:
// . ansi_chars: tableau des caracteres ansi utilises
// . colliding_utf8_chars: tableau des caracteres utf8 initiaux utilises dans windows1252/iso8859-1.
boolean Close();
Expand Down
3 changes: 2 additions & 1 deletion src/Learning/KWData/KWClass.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2663,6 +2663,7 @@ boolean KWClass::CheckNativeComposition(boolean bCheckKeys, boolean bVerboseChec
{
NumericKeyDictionary nkdComponentClasses;

// On passe en parametre le dictionnaire nkdComponentClasses de classes traitees, pour detecter les cycles
return InternalCheckNativeComposition(bCheckKeys, bVerboseCheckKeys, NULL, &nkdComponentClasses);
}

Expand Down Expand Up @@ -2763,7 +2764,7 @@ boolean KWClass::InternalCheckNativeComposition(boolean bCheckKeys, boolean bVer
// La cle de la classe utilisee doit etre strictement plus longue que celle de la classe utilisante
// dans le cas d'un lien de composition multiple a trois niveau (ou plus)
// Exception dans le cas d'une classe sans-cle pouvant etre issue d'une regle de creation de table,
// auquel cas les cles ne sont pas necessaire y compris dans le cas d'un flocon creer par des regles
// auquel cas les cles ne sont pas necessaires y compris dans le cas d'un flocon cree par des regles
else if (parentClass != NULL and parentAttribute->GetType() == KWType::ObjectArray and
parentClass->GetKeyAttributeNumber() > 0 and
parentClass->GetKeyAttributeNumber() >= GetKeyAttributeNumber())
Expand Down
2 changes: 1 addition & 1 deletion src/Learning/KWData/KWContinuous.h
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ class KWContinuous : public Object
// Test de validite du numero d'erreur
static boolean CheckError(int nValue);

// Libelle d'erreur (en anglais) associe a un diagnostique de conversion
// Libelle d'erreur (en anglais) associe a un diagnostic de conversion
static const ALString ErrorLabel(int nError);

// Methode utilitaire pour tranformer le separateur decimal ',' en '.'
Expand Down
2 changes: 1 addition & 1 deletion src/Learning/KWData/KWDerivationRuleOperand.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,7 @@ boolean KWDerivationRuleOperand::CheckCompleteness(const KWClass* kwcOwnerClass)
}

// Verification eventuelle de l'attribut si la classe de scope est correcte
// On effectue cette verification meme en cas d'erreur, pour avoir un diagnostique potentiellement plus precis
// On effectue cette verification meme en cas d'erreur, pour avoir un diagnostic potentiellement plus precis
if (bResult and GetOrigin() == OriginAttribute)
{
assert((GetScopeLevel() == 0 and scopeClass == kwcOwnerClass) or
Expand Down
2 changes: 1 addition & 1 deletion src/Learning/KWData/KWMTDatabase.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -573,7 +573,7 @@ boolean KWMTDatabase::CheckPartially(boolean bWriteOnly) const
// On commence par verifier que la classe est stockable sur un systeme a base de cle
if (not mainClass->IsKeyBasedStorable())
{
// Affichage des diagnostiques d'erreur sur les problemes lies au cles
// Affichage des diagnostics d'erreur sur les problemes lies au cles
bOk = mainClass->CheckKeyBasedStorability();
assert(not bOk);

Expand Down
2 changes: 1 addition & 1 deletion src/Learning/KWData/KWMTDatabase.h
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ class KWMTDatabase : public KWDatabase
// et ne doit toujours etre synchronise
mutable ObjectArray oaMultiTableMappings;

// Warnings pour le cas des tables externes non utilises, gardes pour generer des warnings lors du Check
// Warnings pour le cas des tables externes non utilisees, gardees pour generer des warnings lors du Check
//
// Ce cas arrive si la table principale est une table secondaire d'une table externe qu'elle reference.
// Par exemple, dans le cas d'un schema de molecules avec des atomes et des liaisons, les atomes et les liaisons
Expand Down
2 changes: 1 addition & 1 deletion src/Learning/KWDataPreparation/KWDataGrid.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -2302,7 +2302,7 @@ KWDGAttribute::~KWDGAttribute()
delete innerAttributes;
}

// Reinitialisation en mode debug, pour faciliter le diagnostique
// Reinitialisation en mode debug, pour faciliter le diagnostic
debug(dataGrid = NULL);
debug(nAttributeType = KWType::Unknown);
debug(nAttributeIndex = -1);
Expand Down
6 changes: 4 additions & 2 deletions src/Learning/KWUserInterface/KWClassManagementActionView.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -324,10 +324,10 @@ void KWClassManagementActionView::RefreshHelpLists()
// Liste des dictionnaires charges en memoire, par defaut non visible
classNameHelpList = cast(UIList*, GetFieldAt("ClassNames"));

// On commence vider la liste
// On commence par vider la liste
classNameHelpList->RemoveAllItems();

// Collecte de classe stockables ou non a partir du domaine courant,
// Collecte des classes stockables ou non a partir du domaine courant
for (i = 0; i < KWClassDomain::GetCurrentDomain()->GetClassNumber(); i++)
{
kwcClass = KWClassDomain::GetCurrentDomain()->GetClassAt(i);
Expand All @@ -354,6 +354,8 @@ void KWClassManagementActionView::RefreshHelpLists()
// Parametrage des classes non stockables ensuite, avec ligne blanche de separation
if (svNonStorableClassNames.GetSize() > 0)
{
// Il doit y avoir au moins une classe stockable, et donc au moins deux items dans
// la liste en cours, ce qui fait qu'on n'aura jamais deux items vides de suite
assert(svClassNames.GetSize() > 0);
assert(classNameHelpList->GetItemNumber() >= 2);
classNameHelpList->AddItem();
Expand Down
4 changes: 2 additions & 2 deletions src/Norm/base/CommandFile.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -243,9 +243,9 @@ void CommandFile::CloseInputCommandFile()
return;
}

// Lecture si necessaire de la fin du fichier pour avoir des diagnostiques d'erreur complets
// Lecture si necessaire de la fin du fichier pour avoir des diagnostics d'erreur complets
// - bloc if non termine
// - diagnostique sur les cles du parametrage json non utilisees dans les commandes
// - diagnostic sur les cles du parametrage json non utilisees dans les commandes
bIsParserOkBeforeEnd = bParserOk;
if (GetInputParameterFileName() != "" and IsInputCommandFileOpened() and not IsInputCommandEnd() and
nParserLineIndex > 0 and bParserOk)
Expand Down
2 changes: 1 addition & 1 deletion src/Norm/base/MemoryManager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1056,7 +1056,7 @@ void SegPrintStats(MemSegment* self)

#ifndef NOMEMCONTROL

// Methodes de diagnostique des problemes memoire
// Methodes de diagnostics des problemes memoire
void MemPrintStat(FILE* fOutput);
void MemCompleteCheck(FILE* fOutput);

Expand Down
6 changes: 3 additions & 3 deletions src/Norm/base/MemoryManager.h
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
// la librairie dediee baseONM pour beneficier du nouvel allocateur
// en mode release.
//
// En mode debug, un diagnostique de la memoire est realise au fur et a
// En mode debug, un diagnostic de la memoire est realise au fur et a
// mesure des allocations, ce qui permet de detecter la plupart des erreurs
// suivantes de gestion de la memoire:
// Un compte rendu d'utilisation est affiche apres la fin du main.
Expand Down Expand Up @@ -116,7 +116,7 @@ static class PrivateHeapInitializer
// l'allocation du ieme bloc memoire
// Permet en se servant du debuger et en mettant un point
// d'arret sur GlobalExit() de reperer le lieu d'allocation
// d'un bloc non desaloue (diagnostique par MemCompleteCheck())
// d'un bloc non desaloue (diagnostic par MemCompleteCheck())
// Par defaut, 0 signifie pas d'arret
void MemSetAllocIndexExit(longint lAllocIndex);
longint MemGetAllocIndexExit();
Expand Down Expand Up @@ -235,7 +235,7 @@ longint MemGetTotalHeapRequestedMemory();
void MemPrintHeapStats(FILE* fOutput);

////////////////////////////////////////////////////////////////////////////////////////
// Methodes avancees pour diagnostiquement finement la consommation memoire
// Methodes avancees pour diagnostiquer finement la consommation memoire
// Attention, ces methodes ont potentiellement un impact limite, mais potentiellement
// non negligeable sur les performances, si le handler doit etre appele frequemment
// Elles ne doivent etre utilisee pour le suivi fin de l'utilisation de l'allocateur
Expand Down

0 comments on commit 110b87f

Please sign in to comment.