Comments (6)
Hi Johan,
i) True, the print should be a standard native function... The tricky part is to handle the concatenation such as println("dd" + res), it's probably related to a better support of the string type/operation.
ii) Personnaly, I like the current syntax and I'm not really fond of changing all applications due to a new one... @mraulet I'm not sure about the standardization state of this.
iii) Same idea, is there really an interest to change again if it solves the problem ?
iv) This is a bug that has to be fixed ;)
v,vi) Why not... I've never seen any String declaration in the applications :)
And, of course, contributions are always welcome !
Regards,
Hervé
from orcc.
Hi Hervé, All,
i) True, the print should be a standard native function... The tricky part is to handle the concatenation such as println("dd" + res), it's probably related to a better support of the string type/operation.
To support string concatenation is one possible solution. However I am weary of introducing a Java style string concatenation we int, floats etc is magically converted to string values. I think we should start with just "string" + "string", and introduce some helper function "int2str" etc.
ii) Personnaly, I like the current and I'm not really fond of changing all applications due to a new syntax... @mraulet I'm not sure about the standardization state of this.
iii) Same idea, is there really an interest to change again if it solves the problem ?Well, I think we all should be interested if there is a better way. And in any situation applications that are using non standard features are of course in danger of having to be modified. You are the guys driving the standard so I assume you value standard compliance.
iv) This is a bug that has to be fixed ;)
v,vi) Why not... I've never seen any String declaration in the applications :)And, of course, contributions are always welcome !
Great, should I fix the grammar right away?
Cheers,
J
from orcc.
Hahaha, it's great to see your motivation... Be free to make your own branch, we have to wait two things before merging it to the master :
- The reaction of other people (At least, the one of @mraulet )
- We plan to do a release today, It would be preferable to keep the FE in its current state for this one
from orcc.
iv) has been fixed in 6d68080, but only in a branch for now. This better implementation of CAL standard introduces a lot of errors across all projects in https://github.com/orcc/orc-apps repository. These errors need to be fixed before merging this branch to master.
from orcc.
I) print is not part of the standard. So what you propose might be the best solution
We like the java like print. It works well for us.
Ii) it is already c style array declaration in the standard. This involves to change 3 different standards.
Iii) this can be a contribution to MPEG. But at the moment we might have to change the applications according to the syntax allowed in the standard.
Iv) grammar fixed. Works need to be done on applications.
Vi) I have to check he standard. Don t remember the exact syntax
from orcc.
6d68080 has been merged to master (see 73e953b), so iv) & v) are now fixed
from orcc.
Related Issues (20)
- Nesting networks: "ClassCastException: ..PortImpl cannot be cast to ...Instance" HOT 7
- HLS VHDL code gen problem: "mem_size => )" HOT 1
- Bug report for C Backend - C compiler error 2143 HOT 3
- getSizeInBits() method in TypeImpl HOT 3
- Supporting more than 64 bit literals HOT 3
- Eclipse plugin doesn't initialize .xdf file when added using New-File menu HOT 3
- Eclipse parser should enforce type safety when adding values to arrays HOT 1
- (C backend) minting 16 or more short's by prefixing `i` creates potentially illegal C code HOT 2
- C backend incorrectly computing 32bit/64bit integer values in the absence of casting HOT 4
- net.sf.orcc.cal.cal.<type> cannot be resolved to a type. HOT 6
- Mixing of indices of lvalue when using `for` generators in rvalue HOT 2
- inst.getActor() contains no incoming or outgoing port map
- Xtext editor not showing error if action is misspled or missing HOT 1
- xtext lexer warnings
- [C Backend] Fan-out buffer size HOT 4
- Eclipse install issues HOT 2
- [SOLVED] does not appear to work in eclipse 4.7 HOT 2
- [SOLVED] no dialog raised on Set/Update Refinement eclipse 4.7 HOT 1
- [SOLVED] No Actor displayed in Set/Update Refinement dialog in Eclipse 4.3 HOT 1
- which version can work with recent eclipse?
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from orcc.