Skip to content
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.

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

Error with using min as component name #697

Closed
modelica-trac-importer opened this issue Jan 14, 2017 · 3 comments
Closed

Error with using min as component name #697

modelica-trac-importer opened this issue Jan 14, 2017 · 3 comments
Assignees
Labels
enhancement New feature or enhancement L: Blocks Issue addresses Modelica.Blocks P: high High priority issue wontfix Issue that will not be fixed
Milestone

Comments

@modelica-trac-importer
Copy link

Reported by wischhusen on 6 Feb 2012 08:47 UTC
Dragging the Modelica.Blocks.Math.Min (similar with Max) block into a model yields min as component name. This is in my opinion a restricted identifier (as "if" or something else). Errors occur when using min operator in equation of the top-level model (see attached file).

The default name of the block should be modified to something uncritical (like min1 or something else).


Migrated-From: https://trac.modelica.org/Modelica/ticket/697

@modelica-trac-importer modelica-trac-importer added this to the Design74 milestone Jan 14, 2017
@modelica-trac-importer modelica-trac-importer added enhancement New feature or enhancement L: Blocks Issue addresses Modelica.Blocks P: high High priority issue labels Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Comment by dietmarw on 6 Feb 2012 14:19 UTC
I think since this is an easy one to fix, i.e., setting the defaultComponentName to "min1" and "max1" this should be fixed for MSL 3.2.1 without the need of further discussion.

@modelica-trac-importer modelica-trac-importer modified the milestones: MSL3.2.1, Design74 Jan 14, 2017
@modelica-trac-importer
Copy link
Author

Comment by sjoelund.se on 7 Feb 2012 14:10 UTC
min is not a restricted name and there should be no problem with using it except on the top level (but you cannot have a diagram on the top level). if is quite different since it is a keyword (a class cannot be called if, so there is no need to fix anything here).

@modelica-trac-importer
Copy link
Author

Comment by wischhusen on 22 May 2012 11:42 UTC
Okay, that was obviously a tool bug. Ticket is resolved by tool update.

@modelica-trac-importer modelica-trac-importer added the wontfix Issue that will not be fixed label Jan 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement L: Blocks Issue addresses Modelica.Blocks P: high High priority issue wontfix Issue that will not be fixed
Projects
None yet
Development

No branches or pull requests

2 participants