Skip to content

Commit 3440191

Browse files
authored
Switch renderer factory to using const constructors (#2541)
1 parent b9acd7d commit 3440191

File tree

1 file changed

+13
-4
lines changed

1 file changed

+13
-4
lines changed

lib/src/render/renderer_factory.dart

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,14 +16,19 @@ import 'package:dartdoc/src/render/type_parameters_renderer.dart';
1616
import 'package:dartdoc/src/render/typedef_renderer.dart';
1717

1818
abstract class RendererFactory {
19-
static RendererFactory forFormat(String format) {
19+
const RendererFactory();
20+
21+
/// Retrieves the appropriate [RendererFactory] according to the
22+
/// specified [format]. Currently supports `html` or `md` otherwise
23+
/// throws an [ArgumentError].
24+
factory RendererFactory.forFormat(String format) {
2025
switch (format) {
2126
case 'html':
22-
return HtmlRenderFactory();
27+
return const HtmlRenderFactory();
2328
case 'md':
24-
return MdRenderFactory();
29+
return const MdRenderFactory();
2530
default:
26-
throw ArgumentError('Unsupported format: $format');
31+
throw ArgumentError('Unsupported renderer format: $format');
2732
}
2833
}
2934

@@ -59,6 +64,8 @@ abstract class RendererFactory {
5964
}
6065

6166
class HtmlRenderFactory extends RendererFactory {
67+
const HtmlRenderFactory();
68+
6269
@override
6370
TemplateRenderer get templateRenderer => HtmlTemplateRenderer();
6471

@@ -111,6 +118,8 @@ class HtmlRenderFactory extends RendererFactory {
111118
}
112119

113120
class MdRenderFactory extends RendererFactory {
121+
const MdRenderFactory();
122+
114123
@override
115124
TemplateRenderer get templateRenderer => MdTemplateRenderer();
116125

0 commit comments

Comments
 (0)