Skip to content

Commit fdb5577

Browse files
committed
update todos and rename
1 parent cb2869c commit fdb5577

5 files changed

Lines changed: 20 additions & 19 deletions

File tree

src/resolver_and_composer/ast.ts

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ export type NounPhrase =
2626
}>;
2727
export type Determiner = Readonly<{
2828
determiner: Word;
29+
quantity: Dictionary.Quantity;
2930

30-
// TODO: remove these two
31+
// TODO: remove this
3132
kind: Dictionary.DeterminerType;
32-
quantity: Dictionary.Quantity;
3333
}>;
3434
export type AdjectivePhrase =
3535
| Readonly<{
@@ -52,21 +52,22 @@ export type Complement =
5252
| Readonly<{ type: "adjective"; adjective: AdjectivePhrase }>;
5353
export type Adverb = Readonly<{
5454
adverb: Word;
55+
// TODO: remove this
5556
negative: boolean;
5657
}>;
57-
export type Verb = {
58+
export type VerbAdverb = {
5859
preAdverbs: ReadonlyArray<Adverb>;
5960
verb: Word;
6061
postAdverb: null | Adverb;
6162
};
62-
export type WholeVerb = Readonly<{
63-
modal: null | Verb;
64-
verbs: ReadonlyArray<Verb>;
63+
export type Verb = Readonly<{
64+
modal: null | VerbAdverb;
65+
verbs: ReadonlyArray<VerbAdverb>;
6566
}>;
6667
export type VerbPhrase =
6768
| Readonly<{
6869
type: "simple";
69-
verb: WholeVerb;
70+
verb: Verb;
7071
subjectComplement: null | Complement;
7172
contentClause: null | Clause;
7273
object: null | NounPhrase;

src/resolver_and_composer/composer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ function complement(complement: English.Complement) {
7171
return adjective(complement.adjective, 0);
7272
}
7373
}
74-
function singeVerb(verbAdverb: English.Verb) {
74+
function singeVerb(verbAdverb: English.VerbAdverb) {
7575
const { preAdverbs, verb, postAdverb } = verbAdverb;
7676
const verbPost = verb.word === "can" && postAdverb != null &&
7777
postAdverb.adverb.word === "not"

src/translator_legacy/fixer.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -180,13 +180,13 @@ function fixComplement(complement: English.Complement): English.Complement {
180180
};
181181
}
182182
}
183-
function fixSingleVerb(adverbVerb: English.Verb): English.Verb {
183+
function fixSingleVerb(adverbVerb: English.VerbAdverb): English.VerbAdverb {
184184
return {
185185
...adverbVerb,
186186
preAdverbs: fixMultipleAdverbs(adverbVerb.preAdverbs),
187187
};
188188
}
189-
function fixVerb(verb: English.WholeVerb): English.WholeVerb {
189+
function fixVerb(verb: English.Verb): English.Verb {
190190
return {
191191
modal: mapNullable(verb.modal, fixSingleVerb),
192192
verbs: verb.verbs.map(fixSingleVerb),

src/translator_legacy/phrase.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,7 @@ function preverb(
310310
predicate.verb.type === "simple"
311311
) {
312312
const first = predicate.verb.first;
313-
let predicateVerb: English.Verb;
313+
let predicateVerb: English.VerbAdverb;
314314
switch (first.type) {
315315
case "modal":
316316
return null;

src/translator_legacy/verb.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -24,13 +24,13 @@ export type VerbAccessory = Readonly<{
2424
prepositions: ReadonlyArray<English.Preposition>;
2525
}>;
2626
export type FirstVerb =
27-
| (Readonly<{ type: "modal" }> & English.Verb)
27+
| (Readonly<{ type: "modal" }> & English.VerbAdverb)
2828
| (Readonly<{ type: "non-modal" }> & VerbFormsWithAccessory);
2929
export type PartialSimpleVerb =
3030
& VerbAccessory
3131
& Readonly<{
3232
first: FirstVerb;
33-
rest: ReadonlyArray<English.Verb>;
33+
rest: ReadonlyArray<English.VerbAdverb>;
3434
subjectComplement: null | English.Complement;
3535
forObject: boolean | string;
3636
predicateType: null | "verb" | "noun adjective";
@@ -65,7 +65,7 @@ function addModal(
6565
throw new FilteredError("nested modal verb");
6666
case "non-modal": {
6767
const newRest = nullableAsArray(first)
68-
.map((first): English.Verb => {
68+
.map((first): English.VerbAdverb => {
6969
const { adverbs, presentPlural, negated } = first;
7070
const useVerb = presentPlural === "are" ? "be" : presentPlural;
7171
const preAdverbs = takeNegative ? adverbs : [
@@ -189,7 +189,7 @@ function fromVerbForms(
189189
verbForms: VerbFormsWithAccessory,
190190
perspective: Dictionary.Perspective,
191191
quantity: English.Quantity,
192-
): IterableResult<English.WholeVerb> {
192+
): IterableResult<English.Verb> {
193193
const { negated, adverbs } = verbForms;
194194
const is = verbForms.presentSingular === "is";
195195
const presentSingular = is && perspective === "first"
@@ -203,8 +203,8 @@ function fromVerbForms(
203203
? [pastPlural, verbForms.presentPlural, "do"]
204204
: [pastSingular, presentSingular, "does"];
205205
type Result = Readonly<{
206-
modal: null | English.Verb;
207-
doesNot: null | English.Verb;
206+
modal: null | English.VerbAdverb;
207+
doesNot: null | English.VerbAdverb;
208208
verb: string;
209209
postAdverb: null | English.Adverb;
210210
}>;
@@ -346,7 +346,7 @@ function fromVerbForms(
346346
}
347347
return result.map((
348348
{ modal, doesNot, verb, postAdverb },
349-
): English.WholeVerb => ({
349+
): English.Verb => ({
350350
modal,
351351
verbs: [
352352
...nullableAsArray(doesNot),
@@ -401,6 +401,6 @@ export function verb(
401401
}
402402
}
403403
}
404-
export function noAdverbs(verb: English.Word): English.Verb {
404+
export function noAdverbs(verb: English.Word): English.VerbAdverb {
405405
return { preAdverbs: [], verb, postAdverb: null };
406406
}

0 commit comments

Comments
 (0)