[GitHub] jena issue #202: Formatting rampage
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/202 @afs my laptop is on the fritz, so I will try to find a way to merge this later today, but you are welcome to! :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] jena issue #202: Formatting rampage
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/202 Fixed (reverted) the additional autogen files. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] jena issue #202: Formatting rampage
Github user afs commented on the issue: https://github.com/apache/jena/pull/202 Modified but is generated: JSON_Parser, SPARQLParser10, SPARQLParser11, SSE_ParserCore OpVisitorBase is modified. Various uses of "disjoint" have had the static import removed. I have not checked: testing_framework jena-csv jena-jdbc --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] jena issue #202: Formatting rampage
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/202 @afs Are you cool with this now? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] jena issue #202: Formatting rampage
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/202 Autogen code is now back where it started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] jena issue #202: Formatting rampage
Github user afs commented on the issue: https://github.com/apache/jena/pull/202 The auto-generated code has auto-generated comments to that effect at the top :-) ... which Eclipse then goes and hides :-(. RAT seems to have some built-in detection code because these files don't trigger a validation failure. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] jena issue #202: Formatting rampage
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/202 I will back out the mods to autogen code-- certainly there's no point in tweaking that. I'm still very much learning what is and isn't autogen in ARQ and how it all works. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
