Skip to content

Commit d35dd5c

Browse files
committed
Moved loading of ImageMagick projects to a separate method.
1 parent fd4ed48 commit d35dd5c

File tree

2 files changed

+34
-28
lines changed

2 files changed

+34
-28
lines changed

src/Configs.cpp

Lines changed: 32 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -44,34 +44,7 @@ vector<Config> Configs::load(const Options &options)
4444
loadDirectory(options,L"Dependencies",configs);
4545
loadDirectory(options,L"OptionalDependencies",configs);
4646

47-
if (filesystem::exists(options.rootDirectory + L"ImageMagick"))
48-
{
49-
if (options.isImageMagick7)
50-
{
51-
loadConfig(options,L"MagickCore",L"ImageMagick\\MagickCore",configs);
52-
loadConfig(options,L"MagickWand",L"ImageMagick\\MagickWand",configs);
53-
loadConfig(options,L"oss-fuzz",L"ImageMagick\\oss-fuzz",configs);
54-
}
55-
else
56-
{
57-
loadConfig(options,L"MagickCore",L"ImageMagick\\magick",configs);
58-
loadConfig(options,L"wand",L"ImageMagick\\wand",configs);
59-
}
60-
61-
loadCoders(options,configs);
62-
loadConfig(options,L"Magick++",L"ImageMagick\\Magick++",configs);
63-
loadConfig(options,L"demos",L"ImageMagick\\Magick++\\demo",configs);
64-
loadConfig(options,L"filters",L"ImageMagick\\filters",configs);
65-
loadConfig(options,L"utilities",L"ImageMagick\\utilities",configs);
66-
loadDirectory(options,L"OptionalApplications",configs);
67-
}
68-
69-
if (!options.isImageMagick7)
70-
{
71-
for (auto& config : configs)
72-
config.updateForImageMagick6();
73-
}
74-
47+
loadImageMagick(options,configs);
7548
removeInvalidReferences(options,configs);
7649
validate(options,configs);
7750

@@ -139,6 +112,37 @@ void Configs::loadDirectory(const Options &options,const wstring directory,vecto
139112
}
140113
}
141114

115+
void Configs::loadImageMagick(const Options &options,vector<Config> &configs)
116+
{
117+
if (!filesystem::exists(options.rootDirectory + L"ImageMagick"))
118+
return;
119+
120+
if (options.isImageMagick7)
121+
{
122+
loadConfig(options,L"MagickCore",L"ImageMagick\\MagickCore",configs);
123+
loadConfig(options,L"MagickWand",L"ImageMagick\\MagickWand",configs);
124+
loadConfig(options,L"oss-fuzz",L"ImageMagick\\oss-fuzz",configs);
125+
}
126+
else
127+
{
128+
loadConfig(options,L"MagickCore",L"ImageMagick\\magick",configs);
129+
loadConfig(options,L"wand",L"ImageMagick\\wand",configs);
130+
}
131+
132+
loadCoders(options,configs);
133+
loadConfig(options,L"Magick++",L"ImageMagick\\Magick++",configs);
134+
loadConfig(options,L"demos",L"ImageMagick\\Magick++\\demo",configs);
135+
loadConfig(options,L"filters",L"ImageMagick\\filters",configs);
136+
loadConfig(options,L"utilities",L"ImageMagick\\utilities",configs);
137+
loadDirectory(options,L"OptionalApplications",configs);
138+
139+
if (!options.isImageMagick7)
140+
{
141+
for (auto& config : configs)
142+
config.updateForImageMagick6();
143+
}
144+
}
145+
142146
void Configs::removeInvalidReferences(const Options &options,vector<Config> &configs)
143147
{
144148
for (auto& config : configs)

src/Configs.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,8 @@ class Configs
3939

4040
static void loadDirectory(const Options &options,const wstring directory,vector<Config> &configs);
4141

42+
static void loadImageMagick(const Options &options,vector<Config> &configs);
43+
4244
static void removeInvalidReferences(const Options &options,vector<Config> &configs);
4345

4446
static void validate(const Options &options,const vector<Config> &configs);

0 commit comments

Comments
 (0)