Skip to content
Closed
Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
df851d5
Rename generator class to Adapter
tarrinneal Dec 21, 2022
e40d679
create new generator class and dart subclass
tarrinneal Dec 21, 2022
8c4e2a0
cpp and dart test gen
tarrinneal Dec 21, 2022
bcdbdbc
added files
tarrinneal Dec 21, 2022
31bb701
Adds Generator class to all generators
tarrinneal Dec 21, 2022
06315ce
adds swift
tarrinneal Dec 21, 2022
5675c64
Updates tests to use new Adapter naming scheme
tarrinneal Dec 21, 2022
a319b48
Dart generate methods
tarrinneal Dec 21, 2022
6d7405d
convert all generate functions to use new method
tarrinneal Dec 21, 2022
72a380c
Merge branch 'main' of github.com:flutter/packages into skeleton2
tarrinneal Dec 21, 2022
1e59fef
chagngelog
tarrinneal Dec 21, 2022
0964cd3
remove Generator class fields
tarrinneal Dec 21, 2022
f589da4
move paths to options
tarrinneal Dec 22, 2022
637679d
remove dartTestOptions
tarrinneal Dec 22, 2022
68a23e2
Nits and combines source and header generators
tarrinneal Dec 27, 2022
67282cf
renames Adapter to GeneratorAdapter
tarrinneal Dec 27, 2022
d08606c
Update version number for breaking changes
tarrinneal Dec 27, 2022
90fcb67
nits
tarrinneal Dec 27, 2022
ea0ec6c
more personal nits
tarrinneal Dec 27, 2022
6206bad
Fixes dart header bug
tarrinneal Dec 27, 2022
7c3d35c
add gen files for clarity
tarrinneal Dec 27, 2022
ef0b71a
Merge branch 'main' of github.com:flutter/packages into skeleton2
tarrinneal Dec 27, 2022
8bf2dfb
better field naming
tarrinneal Dec 28, 2022
8c6abf2
better field naming
tarrinneal Dec 28, 2022
93a5d1b
removed unneeded dart test generator
tarrinneal Dec 28, 2022
710d1a1
Add filetype to generator
tarrinneal Dec 28, 2022
d189d11
Adds filetype as field to generatorAdapters
tarrinneal Dec 29, 2022
daae569
Merge branch 'main' of github.com:flutter/packages into skeleton2
tarrinneal Dec 29, 2022
88905e3
Merge branch 'main' of github.com:flutter/packages into skeleton2
tarrinneal Dec 30, 2022
29decb9
nits
tarrinneal Dec 30, 2022
b7abbe4
assert
tarrinneal Jan 3, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions packages/pigeon/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
## 4.2.16

* Creates new Generator classes for each language.

## 4.2.15

* Relocates generator classes. (Reverted)
Expand Down
37 changes: 37 additions & 0 deletions packages/pigeon/lib/cpp_generator.dart
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import 'ast.dart';
import 'functional.dart';
import 'generator.dart';
import 'generator_tools.dart';
import 'pigeon_lib.dart' show Error;

Expand Down Expand Up @@ -64,6 +65,42 @@ class CppOptions {
}
}

/// Class that manages all Cpp header code generation.
class CppHeaderGenerator extends Generator<CppOptions> {
/// Instantiates a Cpp Generator.
CppHeaderGenerator({
required CppOptions languageOptions,
required Root root,
required StringSink sink,
required this.path,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Path to output generated Cpp Header file.
String path;

/// Generates Cpp header files with specified [CppOptions]
@override
void generate() {
generateCppHeader(path, languageOptions, root, sink);
}
}

/// Class that manages all Cpp code generation.
class CppSourceGenerator extends Generator<CppOptions> {
/// Instantiates a Cpp Generator.
CppSourceGenerator({
required CppOptions languageOptions,
required Root root,
required StringSink sink,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Generates Cpp files with specified [CppOptions]
@override
void generate() {
generateCppSource(languageOptions, root, sink);
}
}

String _getCodecSerializerName(Api api) => '${api.name}CodecSerializer';

const String _pointerPrefix = 'pointer';
Expand Down
47 changes: 47 additions & 0 deletions packages/pigeon/lib/dart_generator.dart
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import 'package:yaml/yaml.dart' as yaml;

import 'ast.dart';
import 'functional.dart';
import 'generator.dart';
import 'generator_tools.dart';

/// Documentation comment open symbol.
Expand Down Expand Up @@ -55,6 +56,52 @@ class DartOptions {
}
}

/// Class that manages all Dart code generation.
class DartGenerator extends Generator<DartOptions> {
/// Instantiates a Dart Generator.
DartGenerator({
required DartOptions languageOptions,
required Root root,
required StringSink sink,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Generates Dart files with specified [DartOptions]
@override
void generate() {
generateDart(languageOptions, root, sink);
}
}

/// Class that manages all Dart code generation.
class DartTestGenerator extends Generator<DartOptions> {
/// Instantiates a Dart Generator.
DartTestGenerator({
required DartOptions languageOptions,
required Root root,
required StringSink sink,
required this.dartOutPath,
required this.testOutPath,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Path to output generated Dart file.
String dartOutPath;

/// Path to output generated Test file.
String testOutPath;

/// Generates Dart files with specified [DartOptions]
@override
void generate() {
generateTestDart(
languageOptions,
root,
sink,
dartOutPath: dartOutPath,
testOutPath: testOutPath,
);
}
}

String _escapeForDartSingleQuotedString(String raw) {
return raw
.replaceAll(r'\', r'\\')
Expand Down
29 changes: 29 additions & 0 deletions packages/pigeon/lib/generator.dart
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
// Copyright 2013 The Flutter Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.

import 'ast.dart';

/// A Super class of generator classes.
///
/// This is meant to provide structure and direction for future generator work.
abstract class Generator<T> {
/// Instantiates a Generator.
const Generator({
required this.languageOptions,
required this.root,
required this.sink,
});

///
final T languageOptions;

///
final Root root;

///
final StringSink sink;

/// Generates files for specified language with specified [languageOptions]
void generate();
}
2 changes: 1 addition & 1 deletion packages/pigeon/lib/generator_tools.dart
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import 'dart:mirrors';
import 'ast.dart';

/// The current version of pigeon. This must match the version in pubspec.yaml.
const String pigeonVersion = '4.2.15';
const String pigeonVersion = '4.2.16';

/// Read all the content from [stdin] to a String.
String readStdin() {
Expand Down
17 changes: 17 additions & 0 deletions packages/pigeon/lib/java_generator.dart
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import 'ast.dart';
import 'functional.dart';
import 'generator.dart';
import 'generator_tools.dart';
import 'pigeon_lib.dart' show TaskQueueType;

Expand Down Expand Up @@ -84,6 +85,22 @@ class JavaOptions {
}
}

/// Class that manages all Java code generation.
class JavaGenerator extends Generator<JavaOptions> {
/// Instantiates a Java Generator.
JavaGenerator({
required JavaOptions languageOptions,
required Root root,
required StringSink sink,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Generates Java files with specified [JavaOptions]
@override
void generate() {
generateJava(languageOptions, root, sink);
}
}

/// Calculates the name of the codec that will be generated for [api].
String _getCodecName(Api api) => '${api.name}Codec';

Expand Down
17 changes: 17 additions & 0 deletions packages/pigeon/lib/kotlin_generator.dart
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import 'ast.dart';
import 'functional.dart';
import 'generator.dart';
import 'generator_tools.dart';
import 'pigeon_lib.dart' show TaskQueueType;

Expand Down Expand Up @@ -64,6 +65,22 @@ class KotlinOptions {
}
}

/// Class that manages all Kotlin code generation.
class KotlinGenerator extends Generator<KotlinOptions> {
/// Instantiates a Kotlin Generator.
KotlinGenerator({
required KotlinOptions languageOptions,
required Root root,
required StringSink sink,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Generates Kotlin files with specified [KotlinOptions]
@override
void generate() {
generateKotlin(languageOptions, root, sink);
}
}

/// Calculates the name of the codec that will be generated for [api].
String _getCodecName(Api api) => '${api.name}Codec';

Expand Down
33 changes: 33 additions & 0 deletions packages/pigeon/lib/objc_generator.dart
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

import 'ast.dart';
import 'functional.dart';
import 'generator.dart';
import 'generator_tools.dart';
import 'pigeon_lib.dart' show Error, TaskQueueType;

Expand Down Expand Up @@ -63,6 +64,38 @@ class ObjcOptions {
}
}

/// Class that manages all Objc header code generation.
class ObjcHeaderGenerator extends Generator<ObjcOptions> {
/// Instantiates a Objc Generator.
ObjcHeaderGenerator({
required ObjcOptions languageOptions,
required Root root,
required StringSink sink,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Generates Objc header files with specified [ObjcOptions]
@override
void generate() {
generateObjcHeader(languageOptions, root, sink);
}
}

/// Class that manages all Objc code generation.
class ObjcSourceGenerator extends Generator<ObjcOptions> {
/// Instantiates a Objc Generator.
ObjcSourceGenerator({
required ObjcOptions languageOptions,
required Root root,
required StringSink sink,
}) : super(languageOptions: languageOptions, root: root, sink: sink);

/// Generates Objc files with specified [ObjcOptions]
@override
void generate() {
generateObjcSource(languageOptions, root, sink);
}
}

/// Calculates the ObjC class name, possibly prefixed.
String _className(String? prefix, String className) {
if (prefix != null) {
Expand Down
Loading