search

ddobrev / QtSharp

Mono/.NET bindings for Qt
Other
571 stars 52 forks source link

Add QtCore namespace to dependent namespaces with QtGui. #20

Closed golddranks closed 9 years ago

golddranks commented 9 years ago

Just an ad hoc thing to do, but recently I started to review the errors that are in the generated QtGui.cs. This fixes the first of them, placing the "using QtCore;" directive to the start of the file.

ddobrev commented 9 years ago

I am sorry but I cannot take this patch, not in the master branch. It's a hard-coded hack. I know I already have one of my own - the if check about the name of the module - but it is for testing purposes only because all modules take too long to generate.

golddranks commented 9 years ago

Fair enough :) I know that you suggested to write the thing that would sort the dependencies automatically, but I'd like to get QtGui working manually first - there's still a lot of bugs in the generated source, so I just thought a hack would do for a while. But indeed, it's a hard-coded hack. Well, I'll just keep it in my testing branch.