Hi,

Sean has left the following comment at Add a command-line option that sets an object color temporarily http://www.google-melange.com/gci/task/view/google/gci2012/8126202:


progress!


That's looking like progress! It looks like you're on the right track without a few exceptions. You did hook into the -C option and change the behavior there, so that's good for testing. If the background is still changing, though, you may not be recompiling/testing correctly since you clearly removed the code that called rt_do_cmd() which is what set the background global to the -C color. Put a bu_log() or printf() statement before change_color() (should use consistent spelling) and make sure you're running the right binary when you're testing.

As for the change_color() command, that's a good first step looking at what the matter command does. Note, however, that it's not an acceptable final solution as that method involves updating the comb structure and then WRITING IT OUT TO DISK (via GED_DB_PUT_INTERNAL())... you'll need to find a method that doesn't require read/write access to the .g file and doesn't write to disk. Try to get it to work that way first, though, to make sure you're passing parameters correctly. I don't see you verify that argc/argv are actually geometry names and not other parameters, for example. Print argv to make sure it's actually geometry and not a file name or option while testing.


Greetings,
The Google Open Source Programs Team


---
You are receiving this message because you are subscribed to Add a command-line option that sets an object color temporarily. To stop receiving these messages, go to: http://www.google-melange.com/gci/task/view/google/gci2012/8126202.

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnmore_123012
_______________________________________________
BRL-CAD Tracker mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-tracker

Reply via email to