[sword-devel] I implore you...

[Chris Burrell]

I haven't touched c code for a while but can't you simply use a grep to
exclude the stuff you're not interested in?
On 9 Jun 2013 22:55, "Jaak Ristioja" <jaak at ristioja.ee> wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> On 09.06.2013 23:21, Troy A. Griffitts wrote:
> > I don't think other developers are getting ignored.  Please be
> > specific.  Just because I don't accept a patch doesn't mean a
> > developer is getting ignored.
> >
> > In fact, many times trying to make this release, when people
> > complain that we need something fixed for this release, I ask for a
> > simple testsuite addition to show the problem and desired result,
> > and don't get a response.
> >
> > I don't believe the problem is as you think it is Jaak.  Many
> > people whine about this or that. Not all whine for things to go in
> > the same direction.
> >
> > Everyone whines for a release but not everyone is willing to help
> > submit tests and then fixes for those tests.
> >
> > You stated that you would get involved to help, but you only
> > submit things for which I previously told you I wasn't interested
> > in accepting (worrying about pedantic warnings whose changes often
> > make the code less readable and do nothing to improve any of the
> > real problems for the end user.  Though I do appreciate a few of
> > the warning fixes you submitted, a few being actual bug fixed too
> > (thank you)-- I'm just ranting right now.)
>
> As a BibleTime developer, I want to available tools (-Wall, -Wextra,
> cppcheck, etc) to fix any errors in my code. Due to the Sword header
> files which generate a lot of warnings this task is VERY inconvenient.
> For example, when I compile the whole of BibleTime with GCC, I get 549
> warnings from Sword headers (mostly for unused arguments) - how am I
> supposed to find the warnings relevant for BibleTime? This alone often
> makes it a pain to develop BibleTime and gives me enough reason to
> want to fork Sword.
>
> Turning on and fixing pedantic warnings will help find real bugs.
> FACT! Forcing developers to work blindfolded will not help anyone.
>
> The same tools can be used to find bugs in Sword code, and SHOULD
> regularly be used for this purpose to ensure code quality. As is
> obvious these are currently NOT BEING USED by Sword developers.
> However, when things eventually break, users complain to the BibleTime
> project. Hence, it is also in the interests of front-ends to ensure
> that the code of Sword is of good quality. Again - if Sword won't work
> to ensure this and wont let us in to fix things, we have another
> reason to fork.
>
> This again leads us to the issue of attracting new developers to
> Sword. I don't want to write on this more than necessary to provide a
> small argument for my conclusion. Afaik the current situation isn't
> working well. Biggest obstacles for me personally include working
> blindfolded, submitting patches by e-mail and not getting enough
> feedback for (ignored) patches and other emails.
>
> To conclude - maybe its just me, but altogether I really feel it were
> easier to maintain a parallel fork (at minimal to provide set of
> patches) than to waste my time writing long letters trying to make
> this relationship work in its current form. I accept whatever path the
> Sword project takes, but if it's not enough for the needs of BibleTime
> and our devs, we will make our own choices as well.
>
>
> Blessings,
> Jaak
> The BibleTime team
>
>
> PS: I apologize if this late-night response is incomprehensible.
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v2.0.19 (GNU/Linux)
>
> iQgcBAEBAgAGBQJRtPmyAAoJEEqsYmEt1rCObSdAAIzVPh/6Lsuo2/WhI2/mQWkT
> i1hoxCiM/4kJKgtEkQV2hmsFUvZt+e5gHotdHmOZJghpu/wJdLwITa1hPtl9nIUb
> 3LycH+AWioenSVEm1n12aQlELaY/xYzo/Ev0y9eA1LfPPfrtO3duXTfZrV3Za8mI
> tM5UETdk/Y/JHMNBNjmQTY8k/T8WShyYqmPJ8gDXP2m8mSPsrJjg8j3s5GyNt5iJ
> bLfSp/XePnTmdlG+YiGnszgTc/s9/l3uTYOxDdm8XtRcDkgYx+ihYz5DIl74Z6io
> BCnYoG5M7YnpeOC6HE2j/OtbrefKg2zsYzQfMr4++7HDq9YGiat2rH9s7VMpRvup
> //pFJqeKwMYRN2tZYT/tnbJMCgFbzGT1OMPdy/AWPRn9xhwR37aeEgDvJGMaIrb1
> 4TLstSzO8ZD0BTjKDjhXYkoedZIdZZFnzexa/zqGhxB/p/MJW1NCPCPZrvBlhZZd
> ilXalQIdyMacc6Swtk5cIIh/OlQf/KtNJOg/g3T9wRLTjUIaDFVQE3n86BxBx53a
> BazUa889C6eEuXt00Cgagch5pkJF9ArqhRTZD9PLEgDPgivYfrITiRn1W28xotbc
> WSMiVnb9CdQQ/LyD3MXSz4/lrm4R43wJlRo4jmaTCKrfcXzZA3kYPYrxLKt5a1kT
> inUc5R6fO3UzawoMO72rM2pBA6tv8U3+5G2kF+eSvmV/1PvgOSaJxASOx0qm8WJo
> QWoTRQKmKJOkqkaVKUVLBPhbDQzFOyNYkLLBMKEB2toIjmA/Hdr/G8eEaYe2e37d
> YsgRHZyyLxXnkIag1lUQcTDypTZwjUP8aFGWsKbaYMb8Gyvqf5GpNszMqipG5LSm
> NkTFQgwEUJuO+GcILRDI1tZCby36jEunh5O4e/JeIRMvi/gwxtBMKmErjzThVGk7
> SlRApbeUaYk6sq0V4sHmbvmdQaiFXLQcJ2aRLcw71j7E6FvYQqxLq+BOirzEptmS
> JY5JJwUrQk7Xt7S9H8N45DRgotuK94hyFRo3loAFpLhgTRvjeHlWKNfN+gwVyfph
> 6UuRIdVdMG8pZr/zDRudD3zSAMnjoLb4h0XJBvPDAjx8zIPPzTB96xwkXLnAv3oP
> seTsTM8jCwWOCdg7AWK+QroTjVwM3oCYIjc+Wm0bmn66vYX0UHgsBhGoPt6dK8ov
> 1NzFtvlU9NqvkjFEikmG9QcUZeW8jP8svC/+RJR+D748O6IOxJV8ix1RuCQcM9wL
> u5P27RS8ANq8bOQP5S4cB7e1XzpZgompyTSm73NkaenE1/mGZpU34Nl+VnzqMkgy
> aX/mOF1vsOnmLVy/BJXv/ozHT2fN70bIk3FVbBKaqx+j3msKmX+RBfLmoYyORT2E
> DvSn1wxqSeSq863UcGiIjdASdrylkmC8Q1odiTeqs44B37CzrOEE4dvHFYGR3YB9
> GmQLtAr8f/73ME+o1Ejxg5iWXNaDnBe4vgvBva1hesUnE2O9moGXqcsQUfaMcYe0
> alcm5vvH+WOVn2lL6F5ZFRsOAdia/B0izjPQwbHnthVKaN4HbwAVx5sseov7kEkV
> /QNN8JKTuGkxTNtnR4WTsiZPFAVwqGpUBQply6QqJiI660bnUwTemcIZHSSoscmF
> 9JqZOosEFCKp360t6Jo2c2o5PVtmvQFQl/YPOXaLlmaBodi7mB7Xu1y6bjeIeQEg
> sOEUKc0TTER5wkNqEAUdq1R/CAud9LlLu62V40GK98EvTWo6HtvkGnEbEsueAx9j
> 79/j0uvf1Ro6XdudgMLhgr8nJ8bGrJhY3sy4dS1dcZKoZkILKtxjjemHthJIOjoX
> l6vqkzmh5DibC3IkeTrk+9uS6fIIwS9IdlTtWc/IrzsYezDPQ2kO4BeHkCV3fsgU
> IsbG7d74I487SpZUggvvwdqVg7FuGORD1fkCXrkUjlSplhX1IZ8iO99wnNoVmeeJ
> qmdTMt8gBMQ8y9ap9vyBcZ2d96wHyitKC5Ej4tnJOj9AkSIys11b213kdFo85kFK
> QFE9sAqc9oEgqfx6L3HOeKR1yqdkqH2i2ViNVLxc07QbLLKj/cJ2kpiOlPzYbrcr
> KVBUdvnB3gfTieRpyWTGJ5/nBtj5yHMrIVCcg2PpiLCAL9gN8XpwmHfImkilhfX/
> 0CRQv6xv+gdePJBhGT/dvwlWQHmRCZkwpmH3qJL83EjgHAU4achDNcgUkFNzxLZA
> W8M/mcAOdHvonAr2VhdK5Kpi5mh3N/M4tJPdkLjMAZK0ewvu3TiMb34ugQ1LQraP
> v0p1BBMb3q0jbUIEmw+c973p8hyIRzuS74cBhIExhRrwM1Syo2BAAg7cEMrEahga
> N9+oOkERscRE3bhNjgyeWYuwkZnnBZ2DB979OkqH5GvPQ2MD1ZBUlDlOtHdvOdap
> 9ME+rIXcqr/OTw0p9UP3GZZuzFbzmsqN5HQMH/a46GJm9hsMSsgt4KoGPjl19t2I
> sVzJ2/rtSgDpwpLlVngZuv2jdDkPROWhNYOtpzrjLdyK3mSJVUslNcUosV+rIkia
> 1rIsoRcjlNZDrS6qQa+9aXU8EtysgyovAIDXVrNp+KLvxBvg1uaVQgkLW0cdMBSC
> 8vr3Kbd9qcBvjAg/cC8kjJfltfz0RZs3QdNBOk8/s1BkRh7e0bYBkW1SosZJ9s5a
> RgE/1sib2V1A4C1pTFImwaYwrL1GA9VmksMGSE0IEp7xm0h9OS17t4Mp8EvjVJlM
> wNLd8xTbEAizAa+j1Rr9
> =zSB4
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> sword-devel mailing list: sword-devel at crosswire.org
> http://www.crosswire.org/mailman/listinfo/sword-devel
> Instructions to unsubscribe/change your settings at above page
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.crosswire.org/pipermail/sword-devel/attachments/20130610/cf426bd4/attachment-0001.html>