opened
cnpinto
username_0
Added the minimum amount of changes required for H16. I will do some more testing with H15.0 and the newest release of H16.0, but want to put this up for feedback.

I tested H15.5.632 and H16.0.504.20.qt4 (running the tests locally).

For convenience, here's a list of the major changes to the HDK that I went off of. 
http://www.sidefx.com/docs/hdk16.0/_h_d_k__changes_16_0.html
Add H16 required changes
issue
created
cnpinto
username_0
Is there anything else I can make better?
comment
created
andrewkaufman
username_1
Just gave a few more comments. Once those are addressed, I think we should rebase a bit to get a cleaner history though. That might be a bit tricky with those big "addressed comments" commits, so you might have  to break those apart during the rebase to make them squash more easily.
comment
created
cnpinto
username_0
For copyPrimitive ( const GEO_Primitive *src, GEO_Point **ptredirect ) - the one that isn't used in H14, H15, H16 ), can I keep the function declared but exclude the actual contents of the function and mark them as deprecated? That way it would still compile without having to make any changes to the logic that would affect Houdini < 14 .
comment
created
andrewkaufman
username_1
Sure, sounds good
comment
created
cnpinto
username_0
Okay, I made some more changes. Let me know if there's anything else I can do, and then I'll rebase it with all the changes split up.
comment
created
andrewkaufman
username_1
lets try the rebase
comment
created
cnpinto
username_0
Okay, did the rebase, hopefully that makes sense.
comment
created
cnpinto
username_0
Okay, added those changes to the commits requested for the indirect inputs and primitive geo registration in the plugin.
comment
created
andrewkaufman
username_1
Thanks @username_0 !
comment