thread-based email index, search, and tagging https://git.notmuchmail.org/git/notmuch/atom?h=0.35 2021-09-05T00:07:19Z 2021-09-05T00:07:19Z David Bremner david@tethera.net 2021-08-24T15:17:44Z urn:sha1:4595a814c2117fb33ea39e3ecea41858ad3791a2 We have to rewrite _optimize_tag_query here because it is generating a query string in the infix Xapian syntax. Luckily this is easy to do with the sexp query syntax. 2021-09-05T00:07:19Z David Bremner david@tethera.net 2021-08-24T15:17:10Z urn:sha1:d447b694b498c3e18a22f1eb15139a798c6aab26 It turns out that now that we pass an open database into the subcommands, it is easy to check any requested uuid against the database at the same time as we process the other shared arguments. This results in overall less boilerplate code, as well as making a CLI scope function and variable file scope in notmuch.c. 2021-05-14T09:42:55Z David Bremner david@tethera.net 2021-05-12T23:40:23Z urn:sha1:0c6569d97cda26cf6ce71da86d845d4276967016 This is both a bit clearer and avoids the possibility of modification. 2021-04-07T00:32:36Z David Bremner david@tethera.net 2021-02-27T14:07:45Z urn:sha1:1c747a501c0ed7df5eaa2b994b9ad11d8981c62c At this point it is unused in all subcommands. 2021-03-13T12:45:34Z uncrustify david@tethera.net 2021-03-13T12:45:34Z urn:sha1:eef21c284742fa5ae14d7d352acc3a4dc98821ce This is the result of running $ uncrustify --replace --config devel/uncrustify.cfg *.c *.h in the top level source directory Line breaks were then adjusted manually to keep argc and argv together. 2021-02-06T23:30:20Z David Bremner david@tethera.net 2020-12-07T23:48:48Z urn:sha1:c56dcea7e2da91ada6b6aa737ec115ef4bc91ba6 In addition to changing configuration access, change talloc context for allocation. 2021-02-06T23:05:19Z David Bremner david@tethera.net 2020-12-21T12:55:14Z urn:sha1:319efe21c9d2754c99951cecc71184faf7ca9054 This will allow transitioning individual subcommands to the new configuration framework. Eventually when they are all converted we can remove the notmuch_config_t * argument. For now, live with the parameter shadowing in some some subcommands; it will go away when they are converted. 2017-10-10T01:24:02Z Jani Nikula jani@nikula.org 2017-10-07T08:44:04Z urn:sha1:0f314c0c99befea599a68bea51d759b4133efef6 C99 stdbool turned 18 this year. There really is no reason to use our own, except in the library interface for backward compatibility. Convert the cli and test binaries to stdbool. 2017-10-05T01:00:42Z Jani Nikula jani@nikula.org 2017-10-01T20:53:11Z urn:sha1:4a6721970a42a9f86149fb5d395d1001fed2d305 Several changes at once, just to not have to change the same lines several times over: - Use designated initializers to initialize opt desc arrays. - Only initialize the needed fields. - Remove arg_id (short options) as unused. - Replace opt_type and output_var with several type safe output variables, where the output variable being non-NULL determines the type. Introduce checks to ensure only one is set. The downside is some waste of const space per argument; this could be saved by retaining opt_type and using a union, but that's still pretty verbose. - Fix some variables due to the type safety. Mostly a good thing, but leads to some enums being changed to ints. This is pedantically correct, but somewhat annoying. We could also cast, but that defeats the purpose a bit. - Terminate the opt desc arrays using {}. The output variable type safety and the ability to add new fields for just some output types or arguments are the big wins. For example, if we wanted to add a variable to set when the argument is present, we could do so for just the arguments that need it. Beauty is in the eye of the beholder, but I think this looks nice when defining the arguments, and reduces some of the verbosity we have there. 2017-08-30T10:10:07Z David Bremner david@tethera.net 2017-08-26T14:41:38Z urn:sha1:c960bb4686429b220419f6d45ede920d6c14c3c5 Fix the following cppcheck errors: notmuch-count.c:207: error: Resource leak: input notmuch-tag.c:238: error: Resource leak: input We know that the program is shutting down here, but it does no harm to clean up a bit.