ide: refactoring: use AppendPathDelim/IncludeTrailingPathDelimiter for config paths.

git-svn-id: trunk@51401 -
This commit is contained in:
ondrej 2016-01-24 22:51:50 +00:00
parent befd8e10cb
commit ce45e00b29
9 changed files with 15 additions and 16 deletions

View File

@ -318,7 +318,7 @@ begin
CleanDir(fWorkingDir+PathDelim+'test'); CleanDir(fWorkingDir+PathDelim+'test');
// clean config directory // clean config directory
CleanDir(GetPrimaryConfigPath+PathDelim+'units'); CleanDir(AppendPathDelim(GetPrimaryConfigPath)+'units');
// clean custom target directory // clean custom target directory
if fProfile.TargetDirectory<>'' then begin if fProfile.TargetDirectory<>'' then begin

View File

@ -538,7 +538,7 @@ begin
XMLConfig.GetValue('CodeToolsOptions/Indentation/OnPaste/Enabled',true); XMLConfig.GetValue('CodeToolsOptions/Indentation/OnPaste/Enabled',true);
fIndentationFilename := fIndentationFilename :=
XMLConfig.GetValue('CodeToolsOptions/Indentation/FileName' XMLConfig.GetValue('CodeToolsOptions/Indentation/FileName'
, TrimFilename(GetPrimaryConfigPath + PathDelim +DefaultIndentationFilename)); , TrimFilename(AppendPathDelim(GetPrimaryConfigPath)+DefaultIndentationFilename));
FIndentContextSensitive := FIndentContextSensitive :=
XMLConfig.GetValue('CodeToolsOptions/Indentation/ContextSensitive',true); XMLConfig.GetValue('CodeToolsOptions/Indentation/ContextSensitive',true);
@ -732,7 +732,7 @@ procedure TCodeToolsOptions.SetLazarusDefaultFilename;
var var
ConfFileName: string; ConfFileName: string;
begin begin
ConfFileName:=SetDirSeparators(GetPrimaryConfigPath+'/'+DefaultCodeToolsOptsFile); ConfFileName:=AppendPathDelim(GetPrimaryConfigPath)+DefaultCodeToolsOptsFile;
CopySecondaryConfigFile(DefaultCodeToolsOptsFile); CopySecondaryConfigFile(DefaultCodeToolsOptsFile);
if (not FileExistsCached(ConfFileName)) then begin if (not FileExistsCached(ConfFileName)) then begin
debugln('Looking for code tools config file: "' + ConfFileName + '"'); debugln('Looking for code tools config file: "' + ConfFileName + '"');
@ -899,7 +899,7 @@ begin
FIndentOnLineBreak:=true; FIndentOnLineBreak:=true;
FIndentOnPaste:=true; FIndentOnPaste:=true;
fIndentationFilename:= fIndentationFilename:=
TrimFilename(GetPrimaryConfigPath+PathDelim+DefaultIndentationFilename); TrimFilename(AppendPathDelim(GetPrimaryConfigPath)+DefaultIndentationFilename);
FIndentContextSensitive:=true; FIndentContextSensitive:=true;
// code completion templates // code completion templates

View File

@ -2401,7 +2401,7 @@ end;
function UserSchemeDirectory(CreateIfNotExists: Boolean): String; function UserSchemeDirectory(CreateIfNotExists: Boolean): String;
begin begin
Result := GetPrimaryConfigPath + DirectorySeparator + 'userschemes'; Result := AppendPathDelim(GetPrimaryConfigPath) + 'userschemes';
If CreateIfNotExists and (not DirectoryExistsUTF8(Result)) then If CreateIfNotExists and (not DirectoryExistsUTF8(Result)) then
CreateDirUTF8(Result); CreateDirUTF8(Result);
end; end;
@ -4412,8 +4412,7 @@ begin
inherited Create; inherited Create;
InitLocale; InitLocale;
ConfFileName := SetDirSeparators(GetPrimaryConfigPath + '/' + ConfFileName := AppendPathDelim(GetPrimaryConfigPath) + EditOptsConfFileName;
EditOptsConfFileName);
CopySecondaryConfigFile(EditOptsConfFileName); CopySecondaryConfigFile(EditOptsConfFileName);
try try
if (not FileExistsUTF8(ConfFileName)) then if (not FileExistsUTF8(ConfFileName)) then
@ -4436,7 +4435,7 @@ begin
// code templates (dci file) // code templates (dci file)
fCodeTemplateFileName := fCodeTemplateFileName :=
TrimFilename(GetPrimaryConfigPath+PathDelim+DefaultCodeTemplatesFilename); TrimFilename(AppendPathDelim(GetPrimaryConfigPath)+DefaultCodeTemplatesFilename);
CopySecondaryConfigFile(DefaultCodeTemplatesFilename); CopySecondaryConfigFile(DefaultCodeTemplatesFilename);
if not FileExistsUTF8(CodeTemplateFileName) then if not FileExistsUTF8(CodeTemplateFileName) then
begin begin
@ -4784,7 +4783,7 @@ begin
XMLConfig.GetValue('EditorOptions/CodeTools/AutoDelayInMSec', 1000); XMLConfig.GetValue('EditorOptions/CodeTools/AutoDelayInMSec', 1000);
fCodeTemplateFileName := fCodeTemplateFileName :=
XMLConfig.GetValue('EditorOptions/CodeTools/CodeTemplateFileName' XMLConfig.GetValue('EditorOptions/CodeTools/CodeTemplateFileName'
, TrimFilename(GetPrimaryConfigPath + PathDelim + DefaultCodeTemplatesFilename)); , TrimFilename(AppendPathDelim(GetPrimaryConfigPath) + DefaultCodeTemplatesFilename));
fCTemplIndentToTokenStart := fCTemplIndentToTokenStart :=
XMLConfig.GetValue( XMLConfig.GetValue(
'EditorOptions/CodeTools/CodeTemplateIndentToTokenStart/Value', False); 'EditorOptions/CodeTools/CodeTemplateIndentToTokenStart/Value', False);

View File

@ -433,7 +433,7 @@ var
ConfFileName: String; ConfFileName: String;
begin begin
Result:=nil; Result:=nil;
ConfFileName:=SetDirSeparators(GetPrimaryConfigPath+'/encloseifdef.xml'); ConfFileName:=AppendPathDelim(GetPrimaryConfigPath)+'encloseifdef.xml';
try try
if (not FileExistsUTF8(ConfFileName)) then if (not FileExistsUTF8(ConfFileName)) then
Result:=TXMLConfig.CreateClean(ConfFileName) Result:=TXMLConfig.CreateClean(ConfFileName)

View File

@ -209,7 +209,7 @@ procedure THelpOptions.SetDefaultFilename;
var var
ConfFileName: string; ConfFileName: string;
begin begin
ConfFileName := SetDirSeparators(GetPrimaryConfigPath+'/'+DefaultHelpOptsFile); ConfFileName := AppendPathDelim(GetPrimaryConfigPath)+DefaultHelpOptsFile;
CopySecondaryConfigFile(DefaultHelpOptsFile); CopySecondaryConfigFile(DefaultHelpOptsFile);
if (not FileExistsUTF8(ConfFileName)) then if (not FileExistsUTF8(ConfFileName)) then
DebugLn('NOTE: help options config file not found - using defaults'); DebugLn('NOTE: help options config file not found - using defaults');

View File

@ -80,7 +80,7 @@ var
Config: TConfigStorage; Config: TConfigStorage;
begin begin
if Filename='' then if Filename='' then
Filename:=SetDirSeparators(GetPrimaryConfigPath+'/'+IDEProtocolFilename); Filename:=IncludeTrailingPathDelimiter(GetPrimaryConfigPath)+IDEProtocolFilename;
try try
Config:=DefaultConfigClass.Create(Filename,true); Config:=DefaultConfigClass.Create(Filename,true);
try try
@ -100,7 +100,7 @@ var
Config: TConfigStorage; Config: TConfigStorage;
begin begin
if Filename='' then if Filename='' then
Filename:=SetDirSeparators(GetPrimaryConfigPath+'/'+IDEProtocolFilename); Filename:=IncludeTrailingPathDelimiter(GetPrimaryConfigPath)+IDEProtocolFilename;
try try
Config:=DefaultConfigClass.Create(Filename,false); Config:=DefaultConfigClass.Create(Filename,false);
try try

View File

@ -577,7 +577,7 @@ procedure TInputHistories.SetLazarusDefaultFilename;
var var
ConfFileName: string; ConfFileName: string;
begin begin
ConfFileName:=SetDirSeparators(GetPrimaryConfigPath+'/'+DefaultHistoryFile); ConfFileName:=IncludeTrailingPathDelimiter(GetPrimaryConfigPath)+DefaultHistoryFile;
CopySecondaryConfigFile(DefaultHistoryFile); CopySecondaryConfigFile(DefaultHistoryFile);
FFilename:=ConfFilename; FFilename:=ConfFilename;
end; end;

View File

@ -1063,7 +1063,7 @@ begin
HelpLang := GetLanguageSpecified; HelpLang := GetLanguageSpecified;
if HelpLang = '' then if HelpLang = '' then
begin begin
ConfFileName:=TrimFilename(SetDirSeparators(GetPrimaryConfigPath+'/'+EnvOptsConfFileName)); ConfFileName:=TrimFilename(AppendPathDelim(GetPrimaryConfigPath)+EnvOptsConfFileName);
try try
Cfg:=TXMLConfig.Create(ConfFileName); Cfg:=TXMLConfig.Create(ConfFileName);
try try

View File

@ -222,7 +222,7 @@ var
ConfFileName: string; ConfFileName: string;
begin begin
if fFilename='' then begin if fFilename='' then begin
ConfFileName:=SetDirSeparators(GetPrimaryConfigPath+'/'+MiscOptsFilename); ConfFileName:=AppendPathDelim(GetPrimaryConfigPath)+MiscOptsFilename;
CopySecondaryConfigFile(MiscOptsFilename); CopySecondaryConfigFile(MiscOptsFilename);
if (not FileExistsUTF8(ConfFileName)) then begin if (not FileExistsUTF8(ConfFileName)) then begin
//DebugLn('Note: miscellaneous options file not found - using defaults'); //DebugLn('Note: miscellaneous options file not found - using defaults');