stevedlawrence commented on PR #1058: URL: https://github.com/apache/daffodil/pull/1058#issuecomment-1645824714
Is there much value from having them separate function names? With a goal of simplifying things, what if we changed them all to something like `Assert.assert(...)`. For both the argCheck and stateCheck cases, we are really just doing a bugCheck, so maybe a more generic name would be reasonable for both? Based on some of your comments there are places where usage should be invariant or vice versa so there's clearly some confusing between the different names that this would alleviate. Then the usage() function can be this "public API usage thing", or maybe a new name would be better to make it more clear that it's about incorrect public API usage. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
