Agreed. I'll check on other locations to adjust this behaviour change.<br><br>Jane <br><br>Sent from my Virgin Mobile Android-Powered Device<br><br>----- Reply message -----<br>From: "Tim Tautges" <timothy.tautges@cd-adapco.com><br>To: "Jane Hu" <janejhu@gmail.com><br>Cc: "CGMA Development" <cgma-dev@mcs.anl.gov><br>Subject: [cgma-dev] commit/CGM: Jane Hu: Removed duplicate declaration for iGeom_get_adjacent_entities; Added RefVolume to the list for getting dimension 3 or region info for any RefEntity, per Paul's request. Didn't affect make or make check.<br>Date: Tue, Jun 17, 2014 12:59 pm<br><br><br>Well, if that's the case, why didn't any other code change in this file?  With this change, I expect some of the <br>behavior that uses this function will change, and these changes will be visible to outside apps.  So you've changed <br>behavior to some other arbitrary state, apparently to satisfy one application.  I don't think that's justified.<br><br>- tim<br><br>On 06/17/2014 11:40 AM, Jane Hu wrote:<br>> This is to provide an interface to access ref-volumes in iGeom, it's not supposed to solve the inconsistency issue which<br>> Paul agreed that your suggestions will be the best solution. As Paul mentioned, it should be a through discussion about it.<br>><br>> Jane<br>><br>><br>> On Tue, Jun 17, 2014 at 11:01 AM, Tim Tautges <timothy.tautges@cd-adapco.com <mailto:timothy.tautges@cd-adapco.com>> wrote:<br>><br>>     Wait a minute - how is this solving the general problem?  This looks like a hack to get one application working as<br>>     that app desires, with no improvement in consistency.  What's the reasoning for this change?<br>><br>>     - tim<br>><br>><br>>     On 06/17/2014 10:30 AM, Bitbucket wrote:<br>><br>>         1 new commit in CGM:<br>><br>>         https://bitbucket.org/__fathomteam/cgm/commits/__73512689d31c/<br>>         <https://bitbucket.org/fathomteam/cgm/commits/73512689d31c/><br>>         Changeset:   73512689d31c<br>>         Branch:      trunk<br>>         User:        Jane Hu<br>>         Date:        2014-06-17 17:30:00<br>>         Summary:     Removed duplicate declaration for iGeom_get_adjacent_entities; Added RefVolume to the list for<br>>         getting dimension 3 or region info for any RefEntity, per Paul's request. Didn't affect make or make check.<br>><br>>         Affected #:  1 file<br>><br>>         Repository URL: https://bitbucket.org/__fathomteam/cgm/ <https://bitbucket.org/fathomteam/cgm/><br>><br>>         --<br>><br>>         This is a commit notification from bitbucket.org <http://bitbucket.org>. You are receiving<br>>         this because you have the service enabled, addressing the recipient of<br>>         this email.<br>><br>><br>>     --<br>>     Timothy J. Tautges<br>>     Manager, Directed Meshing, CD-adapco<br>>     Phone: 608-354-1459 <tel:608-354-1459><br>>     timothy.tautges@cd-adapco.com <mailto:timothy.tautges@cd-adapco.com><br>><br>><br>><br>><br>> --<br>> Jane Hu<br>><br>> Asst. Researcher<br>> Dept. of Engineering Physics<br>> UW @ Madison<br>><br>> "And we know that for those who love God, that is, for those who are called according to his purpose, all things are<br>> working together for good." (Romans 8:28)<br><br>-- <br>Timothy J. Tautges<br>Manager, Directed Meshing, CD-adapco<br>Phone: 608-354-1459<br>timothy.tautges@cd-adapco.com<br>