Generated code does not follow c# naming standards

Bug #641411 reported by Manuel de la Peña on 2010-09-17
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
WADL Sharp
Undecided
Manuel de la Peña

Bug Description

The generated code does not follow naming standards. I'm sure there is no need to explain why this is bad.

Related branches

Changed in wadlsharp:
status: New → In Progress
assignee: nobody → Manuel de la Pena (mandel)
Changed in wadlsharp:
milestone: none → 0.2

Manuel, You now have access to the trunk. Go ahead and merge the code. I am not sure if I have time now to review and merge it.

Manuel de la Peña (mandel) wrote :

Sweet, thanks A LOT. I suppose I should tell you what I suddenly started working on you project. I'm currently the person porting Ubuntu One to Windows. As you may know Canonical rolled out Ubuntu SSO that replaces Launchpad login and we require this to work on Windows.

So far you project has the best code I've seen so far on the WADL + C# real so I have started finding "bugs" (I think of them more as to do things) and trying to fix a long the way ;)

I really appreciate your help and the fact that you allow me to get to commit in trunk, thx!

Yeah. From your LP page and your blog, I could make out that you are porting U1 on Windows.
Well, looking at a code/project from a critical angle can actually help a lot. I believe that constructive criticisms help a lot.

BTW there is one more project of mind called WadlAv [ https://launchpad.net/wadlav ] which actually takes the WADL and generates the assembly instead of code. It uses Reflection.Emit , but due to time constraints, I had to shelve it. It is halfway through. This means that every 2 weeks when LP API can change, the application can update itself if it can generate the new assembly from the fresh WADL.

You have access to WadlAv too (through the team cil-advancers)

Changed in wadlsharp:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers