You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucy.apache.org by nw...@apache.org on 2012/04/18 22:35:45 UTC
[lucy-commits] svn commit: r1327671 - /lucy/trunk/clownfish/src/CFCParseHeader.y
Author: nwellnhof
Date: Wed Apr 18 20:35:45 2012
New Revision: 1327671
URL: http://svn.apache.org/viewvc?rev=1327671&view=rev
Log:
Simplify previous commit
Parcel can be derived from exposure.
Modified:
lucy/trunk/clownfish/src/CFCParseHeader.y
Modified: lucy/trunk/clownfish/src/CFCParseHeader.y
URL: http://svn.apache.org/viewvc/lucy/trunk/clownfish/src/CFCParseHeader.y?rev=1327671&r1=1327670&r2=1327671&view=diff
==============================================================================
--- lucy/trunk/clownfish/src/CFCParseHeader.y (original)
+++ lucy/trunk/clownfish/src/CFCParseHeader.y Wed Apr 18 20:35:45 2012
@@ -54,8 +54,8 @@ S_start_class(CFCParser *state, CFCDocuC
}
static CFCVariable*
-S_new_var(CFCParser *state, CFCParcel *parcel, char *exposure, char *modifiers,
- CFCType *type, char *name) {
+S_new_var(CFCParser *state, char *exposure, char *modifiers, CFCType *type,
+ char *name) {
int inert = false;
if (modifiers) {
if (strcmp(modifiers, "inert") != 0) {
@@ -64,9 +64,11 @@ S_new_var(CFCParser *state, CFCParcel *p
inert = true;
}
- const char *class_name = NULL;
+ CFCParcel *parcel = NULL;
+ const char *class_name = NULL;
const char *class_cnick = NULL;
if (exposure && strcmp(exposure, "local") != 0) {
+ parcel = CFCParser_get_parcel(state);
class_name = CFCParser_get_class_name(state);
class_cnick = CFCParser_get_class_cnick(state);
}
@@ -408,28 +410,26 @@ class_defs(A) ::= class_defs(B) subrouti
var_declaration_statement(A) ::=
type(D) declarator(E) SEMICOLON.
{
- A = S_new_var(state, CFCParser_get_parcel(state),
- CFCParser_dupe(state, "parcel"), NULL, D, E);
+ A = S_new_var(state, CFCParser_dupe(state, "parcel"), NULL, D, E);
}
var_declaration_statement(A) ::=
exposure_specifier(B)
type(D) declarator(E) SEMICOLON.
{
- A = S_new_var(state, CFCParser_get_parcel(state), B, NULL, D, E);
+ A = S_new_var(state, B, NULL, D, E);
}
var_declaration_statement(A) ::=
declaration_modifier_list(C)
type(D) declarator(E) SEMICOLON.
{
- A = S_new_var(state, CFCParser_get_parcel(state),
- CFCParser_dupe(state, "parcel"), C, D, E);
+ A = S_new_var(state, CFCParser_dupe(state, "parcel"), C, D, E);
}
var_declaration_statement(A) ::=
exposure_specifier(B)
declaration_modifier_list(C)
type(D) declarator(E) SEMICOLON.
{
- A = S_new_var(state, CFCParser_get_parcel(state), B, C, D, E);
+ A = S_new_var(state, B, C, D, E);
}
subroutine_declaration_statement(A) ::=
@@ -591,7 +591,7 @@ declarator(A) ::= IDENTIFIER(B).
param_variable(A) ::= type(B) declarator(C).
{
- A = S_new_var(state, NULL, NULL, NULL, B, C);
+ A = S_new_var(state, NULL, NULL, B, C);
}
param_list(A) ::= LEFT_PAREN RIGHT_PAREN.