Skip to content

Fix xtext grammar and reference node in process #194

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
ipa-rwu wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

Fix xtext grammar and reference node in process #194

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
92738ab
Basics xtext: fix array format, add ":" after "default"
ipa-rwu May 10, 2023
ae1b0b4
Ros grammar: replace name with plural
ipa-rwu May 10, 2023
3aa5e54
Basic grammer: reformat MessagePart, define msg datatype as [datatype]
ipa-rwu May 10, 2023
7092192
Ros grammar: define msg Spec as list
ipa-rwu May 10, 2023
ac0d82d
Ros grammar: let eclipse generate
ipa-rwu May 10, 2023
796b934
Ros2 grammar: fix grammar, add ':' after type
ipa-rwu May 10, 2023
3437f51
Ros2 grammar: let eclipse generate
ipa-rwu May 10, 2023
d1ce353
Rossystem metamodel: add ComponentRef to reference rosnode in process…
ipa-rwu May 10, 2023
97cafc6
Rossystem metamodel: let eclipse generate
ipa-rwu May 10, 2023
8e6e85b
Rossystem grammar: uncapitalize index, fix process grammar
ipa-rwu May 10, 2023
3436be7
rossystem grammar: let eclipse generate from xtext
ipa-rwu May 10, 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
80 changes: 40 additions & 40 deletions plugins/de.fraunhofer.ipa.ros.xtext/src/de/fraunhofer/ipa/ros/Ros.xtext
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ Package_Impl returns Package:
('fromGitRepo:' fromGitRepo=EString)?
('specs:'
BEGIN
spec+=SpecBase*
PreListElement spec+=SpecBase
(PreListElement spec+=SpecBase)*
END
)?
('dependencies:' '[' dependency+=Dependency (',' dependency+=Dependency)* ']' )?
Expand Down Expand Up @@ -72,44 +73,43 @@ Artifact returns Artifact:

Node returns Node:
'node:' name=RosNames
BEGIN
Copy link
Member

@ipa-nhg ipa-nhg May 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was this "BEGIN" removed intentionally? why?
The interfaces should be defined under the node.

(
('publishers:'
BEGIN
publisher+=Publisher*
END
)|
('subscribers:'
BEGIN
subscriber+=Subscriber*
END
)|
('serviceservers:'
BEGIN
serviceserver+=ServiceServer*
END
)|
('serviceclients:'
BEGIN
serviceclient+=ServiceClient*
END
)|
('actionservers:'
BEGIN
actionserver+=ActionServer*
END
)|
('actionclients:'
BEGIN
actionclient+=ActionClient*
END
)|
('parameters:'
BEGIN
parameter+=Parameter*
END
)
)*END
(
('publishers:'
BEGIN
publisher+=Publisher*
END
)|
('subscribers:'
BEGIN
subscriber+=Subscriber*
END
)|
('serviceServers:'
BEGIN
serviceserver+=ServiceServer*
END
)|
('serviceClients:'
BEGIN
serviceclient+=ServiceClient*
END
)|
('actionServers:'
BEGIN
actionserver+=ActionServer*
END
)|
('actionClients:'
BEGIN
actionclient+=ActionClient*
END
)|
('parameters:'
BEGIN
parameter+=Parameter*
END
)
)*
;

///////////////////
Expand Down Expand Up @@ -146,7 +146,7 @@ ActionSpec returns ActionSpec:

MessageDefinition returns MessageDefinition:
{MessageDefinition}
MessagePart+=MessagePart*;
MessagePart+=MessagePart+;

///////////////////
//INTERFACES
Expand Down