[Monotone-commits-diffs] Revision fec274f8b7d805df3b2d9b8c8bf2ae917565461e

[monotone]

-----------------------------------------------------------------
Revision: fec274f8b7d805df3b2d9b8c8bf2ae917565461e
Ancestor: 66a3773cebd434426e2e3ba85c6ed7e823f4caaa
Author: address@hidden
Date: 2007-04-02T03:28:11
Branch: net.venge.monotone

Modified files:
        app_state.cc cert.cc cert.hh cmd_key_cert.cc cmd_merging.cc
        cmd_packet.cc cmd_ws_commit.cc key_store.cc key_store.hh
        merge.cc packet.cc packet.hh project.cc project.hh
        rcs_import.cc

ChangeLog: 

        * cert.cc: Don't include packet.hh.
        (put_simple_revision_cert): Use db.put_revision_cert.  Remove
        packet_consumer argument.
        (cert_revision_in_branch): Remove packet_consumer argument.
        Take a branch_name, not a cert_value.
        (cert_revision_date_time, cert_revision_author)
        (cert_revision_author_default, cert_revision_tag)
        (cert_revision_changelog, cert_revision_comment)
        (cert_revision_testresult): Remove packet_consumer argument.
        (branch_cert_name, date_cert_name, author_cert_name)
        (tag_cert_name, changelog_cert_name, comment_cert_name):
        Don't define.
        * cert.hh: Update all prototypes.
        (struct packet_consumer): Don't forward-declare.
        (branch_cert_name, date_cert_name, author_cert_name)
        (tag_cert_name, changelog_cert_name, comment_cert_name):
        Define as macros to work around bad code generated by GCC for
        file-scope std::string (or thing-with-a-std::string-inside)
        objects.
        * project.cc: Don't include packet.hh.  Update calls to changed
        cert functions.
        (put_revision_in_branch, put_tag, put_standard_certs)
        (put_standard_certs_from_options, put_cert): Remove packet_consumer
        argument.
        * project.hh: Update all prototypes.

        * cmd_key_cert.cc, cmd_merging.cc, cmd_ws_commit.cc, rcs_import.cc:
        Don't include packet.hh. Update calls to changed cert/project
        functions.  No need for a packet_db_writer in various functions.
        * merge.cc: Don't include packet.hh.
        * packet.cc, packet.hh: Move definition of struct packet_db_writer
        and everything to do with it from here ...
        * cmd_packet.cc: ... to here, above sole remaining user, and put
        in an anonymous namespace.

        * key_store.hh: Prune #includes.  Don't mention keyreader.
        (key_store): keyreader is no longer a friend.  Change app field
        from pointer to reference, and adjust constructor to match.
        (keyreader::put_key_pair_memory): New function.
        * key_store.cc (keyreader): Put in an anonymous namespace.  Change
        ks field from pointer to reference, and adjust constructor to
        match.
        (keyreader::consume_key_pair): Use key_store::put_key_pair_memory.
        (key_store::put_key_pair_memory): Define.
        (key_store::put_key_pair): Use put_key_pair_memory for most of the
        work.
        * app_state.cc: Adjust key_store constructor call.

txt4ZAACgFdtH.txt
Description: Diff [66a3773cebd434426e2e3ba85c6ed7e823f4caaa] -> [fec274f8b7d805df3b2d9b8c8bf2ae917565461e]