NightOwl888 commented on issue #737: URL: https://github.com/apache/lucenenet/issues/737#issuecomment-1300798260
Well, that is because you know what you are doin' :). But those mere mortals who allow a `null` to pass into one of our APIs might be chasing their tail for awhile because the null gets passed down through 5 or so more APIs before it hits the one that throws. A guard clause makes the stack trace much more explicit as to where the problem lies. And who knows how long they will chase their tail if they don't even have a vague notion that they can turn on asserts. > What has been most enlightening about the scan is that it is pointing us to very specific areas that have a small problem, and in several instances it led to finding a much bigger problem in the same area. Even though the scan didn't pick it up the bigger problem as an issue. -- 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: dev-unsubscr...@lucenenet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
