added a Content-Description to the signature part - part of a merge from the 1-0 branch.

merged some changes from the 1-0 branch that apparently never got merged into HEAD

Protect against either of the types being NULL.

2001-12-04  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-utils.c (header_content_type_simple): Protect against
either of the types being NULL.

2001-12-05  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-filter-basic.c (filter): If complete() allocates
len+2 bytes for the out buffer, so should this. See bug #16371 for
an example case.

2001-12-05  Jeffrey Stedfast  <fejj@ximian.com>

* camel-sasl-digest-md5.c: iconv() returns a size_t, not an int.

* camel-pgp-context.c: The return value of iconv() is a size_t,
not an int.

* camel-mime-part-utils.c (convert_buffer): Always use size_t args
for iconv().

* camel-mime-filter-charset.c (complete): Always use size_t args
for iconv().
(filter): Same.

* camel-mime-utils.c (header_address_fold): Make headerlen a
size_t instead of an int.
(header_fold): Same.
(base64_encode_close): We should be returning a size_t and inlen
should also be a size_t.
(base64_encode_step): Same here.
(base64_decode_step): Here too.
(base64_encode_simple): And here...
(base64_decode_simple): Same.
(uuencode_close): We should also use size_t's here...
(uuencode_step): And here too.
(uudecode_step): And also here.
(quoted_encode_close): Same idea here.
(quoted_encode_step): Again here.
(quoted_decode_step): Here too.
(quoted_encode): Input length should be a size_t.
(rfc2047_decode_word): Same.
(g_string_append_len): Here too.
(append_8bit): "
(rfc2047_encode_word): "
(quote_word): "
(hex_decode): "
(rfc2184_decode): Use size_t's with iconv().
(header_decode_param): Same.

Fix the tm_gmtoff case (its sign is the opposite of "timezone"). Fixes

* camel-mime-message.c (camel_mime_message_set_date): Fix the
tm_gmtoff case (its sign is the opposite of "timezone"). Fixes
#14678

Implements marking messages as "Need Reply".

Implements marking messages as "Need Reply".

2001-12-09  Jon Trowbridge  <trow@ximian.com>

* mail-need-reply.xpm: Added a really, really ugly and
awful icon to symbolize "message needs a reply".

* Makefile.am (EXTRA_DIST): Added mail-need-reply.xpm.

2001-12-09  Jon Trowbridge  <trow@ximian.com>

* camel-folder-summary.c: Add "NeedsReply" to the flag_names array
for CAMEL_MESSAGE_NEEDS_REPLY.

* camel-folder-summary.h: Added CAMEL_MESSAGE_NEEDS_REPLY flag.

2001-12-09  Jon Trowbridge  <trow@ximian.com>

* vfoldertypes.xml: Add "Needs Reply" option to different status
types.

* filtertypes.xml: Add "Needs Reply" option to different status
types.

2001-12-09  Jon Trowbridge  <trow@ximian.com>

* message-list.c: #include "art/mail-need-reply.xpm".
(ml_tree_value_at): Adjust magic numbers, show "Need Reply" icon
if the message needs reply.
(message_list_create_extras): Adjust magic numbers to add new
icon.
(on_click): Changed to toggle between unread, read, and need reply
when the status icon is clicked.

* mail-callbacks.c (mark_as_needing_reply): Added.
(mark_as_not_needing_reply): Added.  Add "set" value to struct
post_send_data.
(composer_sent_cb): Use both "flags" and "set" elements of
post_send_data when setting message flags.
(mail_reply): Clear "Needs Reply" flag when we actually reply to a
message.

* folder-browser.c: Changed flag values to be given by
bit-shifting (1<<5) vs. base-ten (32).  Added
CAN_MARK_DOESNT_NEED_REPLY flag.  Added "Mark as Needing Reply"
and "Mark as Not Needing Reply" elements to context menu.
(on_right_click): Hide "Mark as (Not) Needing Reply" context menu
elements as appropriate.

Don't use regex matching. Fixes bug #16227.

2001-11-29  Jeffrey Stedfast  <fejj@ximian.com>

* camel-folder-search.c (search_body_contains): Don't use regex
matching. Fixes bug #16227.

* camel-mime-message.c (best_encoding): Check the content-object's
mime type, not the mime part types. Should fix bug #15843.

2001-11-27  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-folder.c (get_content): Return NULL if
construct_from_stream fails.
(get_message): Same.
(get_message_simple): Here too.
(add_message_from_data): And here.

Use the uid rather than vuid for unmatched. Also add the uid to unmatched

2001-11-14    <NotZed@Ximian.com>

        * camel-vee-folder.c (folder_changed_remove_uid): Use the uid
        rather than vuid for unmatched.  Also add the uid to unmatched if
        it wasn't in the unmatched_uids table at all.
        (folder_changed_change): If checking for added, if the new ones
        dont match, then try to add them to unmatched.  Fixes #6893.

        * camel-folder-summary.c (camel_folder_summary_index): Change lock
        order, always summary_lock before ref_lock.
        (camel_folder_summary_array): "
        (camel_folder_summary_uid): "
        (camel_folder_summary_remove_uid): "  Fixes a deadlock.

Completely new implementation of NNTP.

2001-11-30  Not Zed  <NotZed@Ximian.com>

        * providers/nntp/camel-nntp-*.c:

        Completely new implementation of NNTP.

        Doesn't support subscriptions yet (lists all folders), but should
        be more reliable (faster?), and has an integrated cache.

        * camel-exception.c (camel_exception_new): Use e_memchunks for
        exception blocks.
        (camel_exception_free): Same.

        * camel-data-cache.[ch]: New object for managing on-disk caches of
        anything that can be stored in a camel-stream.

        * camel-file-utils.c (camel_file_util_mkdir): New function, just a
        nicer place to put this (than camel-store), should be removed from
        camel-store.
        (camel_file_util_safe_filename): New function to url-encode a
        filename.

        * camel-mime-parser.c (drop_states): New func to drop the parser
        state to initial state.
        (folder_scan_init_with_fd):
        (folder_scan_init_with_stream): Call above func to reset state if
        the stream is changed on us so we can change streams to reuse a
        parser object.

If the uid doesn't have a ',' in it, fail to crash.

2001-11-25  Not Zed  <NotZed@Ximian.com>

        * providers/nntp/camel-nntp-folder.c (nntp_folder_get_message): If
        the uid doesn't have a ',' in it, fail to crash.

        * providers/nntp/camel-nntp-newsrc.c
        (camel_nntp_newsrc_article_is_read): check group != NULL before scanning.
        (camel_nntp_newsrc_get_highest_article_read): "
        (camel_nntp_newsrc_get_num_articles_read): "
        (camel_nntp_newsrc_mark_range_read): "

        * providers/nntp/camel-nntp-store.c
        (camel_nntp_store_get_overview_fmt): IF we dont have
        nntp_list_follows, dont try and get a list response.
        (nntp_store_get_folder_info): Set path part of folderinfo.

Check to see that errno is non-zero before returning g_strerror. If it's

2001-11-20  Jeffrey Stedfast  <fejj@ximian.com>

* providers/smtp/camel-smtp-transport.c (get_smtp_error_string):
Check to see that errno is non-zero before returning
g_strerror. If it's 0, then we have an unknown error.

For the uudecoding mode, garble up the "begin <mode> <filename>" line

2001-11-26  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-filter-basic.c: For the uudecoding mode, garble up
the "begin <mode> <filename>" line before decoding.

* camel-mime-part-utils.c
(simple_data_wrapper_construct_from_parser): Add a uudecoder if
the transfer encoding is x-uuencode.

* camel-mime-part.c (write_to_stream): Handle x-uuencoded content
too.

Clear the password in bonobo-conf as well. Fixes ximian 14893.

        * e-passwords.c (e_passwords_forget_password): Clear the password
        in bonobo-conf as well. Fixes ximian 14893.

Added a check to see if the operation has been cancelled. (stream_write):

2001-11-19  Jeffrey Stedfast  <fejj@ximian.com>

* camel-tcp-stream-ssl.c (stream_read): Added a check to see if
the operation has been cancelled.
(stream_write): Same.

Updates for compliance with rfc2231

2001-11-19  Jeffrey Stedfast  <fejj@ximian.com>

Updates for compliance with rfc2231

* camel-mime-utils.c (header_encode_param):
camel_mime_special_table[c] & IS_ESAFE should have been
!(camel_mime_special_table[c] & IS_ESAFE). Also added a few
comments for how to improve the code at some future date.
(header_decode_param): Now takes an argument rfc2184_part so our
caller can get this information as well.
(header_decode_param_list): Pass an rfc2184_part argument to
header_decode_param and also added a few comments on where to
improve on rfc2184/rfc2231 compliance.
(rfc2047_decode_word): Updated to respect the updated ABNF syntax
of rfc2047 encoded words, yay.

Abort if body == NULL.

2001-11-16  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-utils.c (imap_parse_body): Abort if
body == NULL.

New mime filter to convert plain text to html.

2001-11-16  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-filter-tohtml.c (camel_mime_filter_tohtml_new): New
mime filter to convert plain text to html.

see the ChangeLog, I forget

If we get a BYE response, call camel_service_disconnect() and set an

2001-11-14  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-command.c
(camel_imap_command_response): If we get a BYE response, call
camel_service_disconnect() and set an exception. Also do the check
for "* BYE" first instead of passing it off imap_read_untagged()
since we'll just waste time in there mallocing left and right only
to arrive at the single response line "* BYE" again :-)

If we have no source string, then use undefined, which should always

2001-11-13    <NotZed@Ximian.com>

        * camel-filter-search.c (get_source): If we have no source string,
        then use undefined, which should always evaluate to FALSE.  Fix
        for #15267.

Added a g_return_val_if_fail for folder_name != NULL.

2001-11-12  Jeffrey Stedfast  <fejj@ximian.com>

* camel-store.c (camel_store_get_folder): Added a
g_return_val_if_fail for folder_name != NULL.

If any of the pipe()'s fail, clean up any pipes that may have succeeded.

2001-11-08  Jeffrey Stedfast  <fejj@ximian.com>

* camel-pgp-context.c (crypto_exec_with_passwd): If any of the
pipe()'s fail, clean up any pipes that may have succeeded. Also
close the password fds.

If we get a user cancel and that causes us to fail to authenticate, abort

2001-11-09    <NotZed@Ximian.com>

        * providers/imap/camel-imap-store.c (imap_auth_loop): If we get a
        user cancel and that causes us to fail to authenticate, abort
        rather than loop forever.  Fix for #14951.

Now takes a command-length argument so we can 1) avoid duping the command

2001-11-06  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-command.c
(camel_imap_command_continuation): Now takes a command-length
argument so we can 1) avoid duping the command string yet again,
yay. 2) we now don't have to worry about embedded nul-chars
screwing us over (we still need to avoid allowing them into the
string but at least now it won't mess us up).

* providers/imap/camel-imap-folder.c (do_append): Instead of
appending a nul char to the end of the byte array and then passing
that off as if it were a string to
camel_imap_command_continuation, instead pass the byte-array
length since that function now takes a length argument. Yay.  Also
encode any 8bit parts to avoid the possibility of sending embedded
nul chars to the imap server.

* providers/imap/camel-imap-store.c (try_auth): Updated to pass a
command-length argument to camel_imap_command_continuation().

Oops, pass a mode argument to the open() call.

2001-11-07  Jeffrey Stedfast  <fejj@ximian.com>

* camel-tcp-stream-ssl.c (save_ssl_cert): Oops, pass a mode
argument to the open() call.

* camel-tcp-stream-openssl.c (save_ssl_cert): Oops, pass a mode
argument to the open() call.

Add code to kludge around any x-inline-pgp-hacks by prepending some

2001-11-07  Jeffrey Stedfast  <fejj@ximian.com>

* camel-pgp-mime.c (camel_pgp_mime_part_decrypt): Add code to
kludge around any x-inline-pgp-hacks by prepending some content
headers to the decrypted data.

Initialize our private mutex. (camel_data_wrapper_finalize): Destroy our

2001-11-02  Jeffrey Stedfast  <fejj@ximian.com>

* camel-data-wrapper.c (camel_data_wrapper_init): Initialize our
private mutex.
(camel_data_wrapper_finalize): Destroy our private mutex.
(write_to_stream): Lock around camel_stream_write_to_stream() so
that we don't get 2 threads trying to read from our stream at the
same time.

Move all searching operations outside of any locks except subfolder lock,

2001-11-01    <NotZed@Ximian.com>

        * camel-vee-folder.c (folder_changed_change): Move all searching
        operations outside of any locks except subfolder lock, same as
        build_folder.  Fix for #14294.

Same as below.

2001-10-31    <NotZed@Ximian.com>

        * providers/local/camel-spool-folder.c (spool_get_message): Same
        as below.

        * providers/local/camel-maildir-folder.c (maildir_get_message):
        Same as below.

        * providers/local/camel-mbox-folder.c (mbox_get_message): Set
        USER_CANCEL if failed due to EINTR.

        * camel-filter-driver.c (camel_filter_driver_filter_mbox): If
        construct from parser fails due to user cancel, set USER_CANCEL on
        exception.

        * camel-mime-part.c (construct_from_parser): Return error if the
        parser had an io error.

        * camel-mime-message.c (construct_from_parser): Check error on
        parser/return error.

        * camel-mime-parser.c (folder_scan_init): Init error number.
        (camel_mime_parser_errno): New function, return errno of any io
        failures.
        (folder_read): Set errno if a failure occured.
        (folder_seek): Same.
        (folder_scan_init_with_fd): Setup errno depeding on ok/failure.
        (folder_scan_init_with_stream): Same.

Dont setup cancel_fd. (camel_operation_cancel_fd): If cancel_fd not

2001-10-30    <NotZed@Ximian.com>

        * camel-operation.c (camel_operation_new): Dont setup cancel_fd.
        (camel_operation_cancel_fd): If cancel_fd not created, set it up.
        (camel_operation_cancel): Set cancel flag before sending message.
        (cancel_thread): Same.

Make sure we unref the mimeparse when we're done (successfully). This was

2001-10-30    <NotZed@Ximian.com>

        * camel-filter-driver.c (camel_filter_driver_filter_mbox): Make
        sure we unref the mimeparse when we're done (successfully).  This
        was leaking an fd every get-mail! :(

        * camel-lock-client.c (camel_lock_helper_init): Close all fd's
        above 2.

Remove the X-Evolution header before anyone else gets to see this private

2001-10-30    <NotZed@Ximian.com>

        * providers/local/camel-mbox-folder.c (mbox_get_message): Remove
        the X-Evolution header before anyone else gets to see this private
        thing.  Shoudl fix 11676 since we were inheriting flags from
        X-Evolution headers we didn't want to.

If we can't retrieve the message, ignore it, it can't match.

2001-10-30    <NotZed@Ximian.com>

        * camel-folder-search.c (match_message): If we can't retrieve the
        message, ignore it, it can't match.

        * providers/imap/camel-imap-search.c
        (camel_imap_search_class_init): Setup parent class pointer.
        (imap_body_contains): If offline, just use the parent
        body_contains method which will get the messages and search them
        manually.

        * providers/imap/camel-imap-folder.c (imap_search_by_expression,
        imap_search_by_uids): Remove offline check.

Fix this to match get_headers.

* camel-mime-part.c (free_headers): Fix this to match get_headers.

Dont try to move ibex if we have none.

2001-10-30    <NotZed@Ximian.com>

        * providers/local/camel-local-store.c (rename_folder): Dont try to
        move ibex if we have none.

Fixed compilation errors.

2001-10-30    <NotZed@Ximian.com>

        * camel-tcp-stream-openssl.c (stream_write): Fixed compilation
        errors.

Removed unused variable.

2001-10-29  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-utils.c (camel_mime_utils_init): Removed unused
variable.

fixed a compile problem and saved errno before calling fcntl() in stream_read

w(x) out some warnings.

2001-10-30    <NotZed@Ximian.com>

        * camel-mime-utils.c (header_decode_mailbox): w(x) out some warnings.

        * providers/local/camel-local-summary.c
        (camel_local_summary_load): Remove the warning about not loading
        summary file - its a valid case for new folders.

Removed. (stream_write): Keep looping (non-blocking case) if errno is

2001-10-29  Jeffrey Stedfast  <fejj@ximian.com>

* camel-tcp-stream-openssl.c (my_SSL_write): Removed.
(stream_write): Keep looping (non-blocking case) if errno is
EAGAIN, EINTR or EWOULDBLOCK. For NONBLOCKing I/O, sync up with
CamelTcpStreamRaw. As with CamelTcpStreamRaw/SSL - make sure to
write out everything before returning.
(my_SSL_read): Removed.
(stream_read): Just call ssl_error_to_errno() and check the errno
values that we care about so we can keep the general look of all
this stream code the same. Also when checking the return value of
SSL_read, check for <0 instead of ==-1 since the man page for
SSL_read doesn't say it will return -1 on fail, it just says <0.
(stream_flush): Don't fsync() since syncing on a socket is a Bad
Thing (tm).

* camel-tcp-stream-ssl.c (stream_write): Make sure we write out
everything just like in camel-tcp-stream-raw.c.

* camel-stream-buffer.c (camel_stream_buffer_gets): If
camel_stream_read() returns -1, don't necessarily return -1 to our
caller since it's possible that we did actually "read" some data
(ie, we copied some pre-buffered data into the out buffer).

* camel-stream-buffer.h: Removed CAMEL_STREAM_BUFFER_NEWLINE since
it never got used anywhere and it isn't supported anyway.

If we get an empty name, then mark that as NoSelect. Workaround fix for

2001-10-30    <NotZed@Ximian.com>

        * providers/imap/camel-imap-store.c
        (parse_list_response_as_folder_info): If we get an empty name,
        then mark that as NoSelect.  Workaround fix for #13239.

Set an X-Evolution-Source header on the returned message so replies come

* providers/imap/camel-imap-folder.c (get_message_simple,
imap_get_message): Set an X-Evolution-Source header on the
returned message so replies come from the right identity.

If we're not going to search on a changed uid, make sure we change out

2001-10-29    <NotZed@Ximian.com>

        * camel-vee-folder.c (folder_changed_change): If we're not going
        to search on a changed uid, make sure we change out copy still.
        Fixes #13916.

Don't emit a folder_subscribed signal if we are in the process of renaming

2001-10-29  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-store.c (subscribe_folder): Don't emit
a folder_subscribed signal if we are in the process of renaming
folders.
(unsubscribe_folder): Same here but for the unsubscribe signal.
(rename_folder): Unsubscribe from the folder being renamed and any
subfolders it may have before actually renaming. Once the folder
has been renamed, re-subscribe to it and it's subfolders with the
new name.

* camel-store.c (camel_folder_info_build): Sort the folder info's
before constructing the tree.

Minor fix - should have been using a strNcmp when checking if it was an

2001-10-29  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-utils.c (header_decode_param): Minor fix - should
have been using a strNcmp when checking if it was an rfc2047
encoded word.

Notify the user that PGP 2.6.x is no longer supported and use

2001-10-29  Jeffrey Stedfast  <fejj@ximian.com>

* camel-pgp-context.c (pgp_sign): Notify the user that PGP 2.6.x
is no longer supported and use camel_exception_set() instead of
setv() where appropriate.
(pgp_clearsign): Same.
(pgp_verify): And again here.
(pgp_encrypt): Here too.
(pgp_decrypt): And finally here.

Don't free the hash table key if the folder wasn't found in the hash

* providers/imap/camel-imap-store.c (unsubscribe_folder): Don't
free the hash table key if the folder wasn't found in the hash
table. I'm not sure where the bug is/was that made it possible to
add a folder to the tree after creating it failed, but now if that
happens, it won't crash if you try to delete it again. (#11492)

if the service is being connected and thinks it's online, but the session

* camel-disco-store.c (camel_disco_store_status): if the service
is being connected and thinks it's online, but the session is
offline, then mark the service offline as well. Fixes 13683.

Turn off w() -> warnings.

2001-10-29    <NotZed@Ximian.com>

        * camel-mime-utils.c: Turn off w() -> warnings.

        * providers/imap/camel-imap-store.c (rename_folder): Also rename
        the message cache.
        (rename_folder): Fix subscribed folders table with rename(s).

        * providers/imap/camel-imap-folder.c (imap_rename): Implement,
        rename the cache dir pointer.

        * providers/imap/camel-imap-message-cache.c
        (camel_imap_message_cache_set_path): New method to set the path
        used by a message cache.

Fix the 'is this a matching subfolder or subfolder'' logic to match

2001-10-29    <NotZed@Ximian.com>

        * camel-vee-store.c (build_info): Fix the 'is this a matching
        subfolder or subfolder'' logic to match camel-store
        get_subfolders'.

        * camel-store.c (get_subfolders): Fix the 'is this a matching
        folder or subfolder' logic.

Changed to call ibex_move to rename it internally.

2001-10-28    <NotZed@Ximian.com>

        * providers/local/camel-local-store.c (rename_folder): Changed to
        call ibex_move to rename it internally.

Use the right variable, not info.

2001-10-28    <NotZed@Ximian.com>

* camel-store.c (camel_store_rename_folder): Use the right variable, not info.

Make sure we've actually read data before checking if p[-] is '\r' or not.

2001-10-28  Jeffrey Stedfast  <fejj@ximian.com>

* camel-stream-buffer.c (camel_stream_buffer_read_line): Make sure
we've actually read data before checking if p[-] is '\r' or not.

Same.

2001-10-28  Jeffrey Stedfast  <fejj@ximian.com>

* camel-tcp-stream-raw.c (stream_write): Same.

* camel-stream-fs.c (stream_write): If errno isn't EAGAIN, just
exit with -1 and instead of using continue (which doesn't go to
the top of the do-while thus making our logic suck), just set w to
0. Still seems to be a bug in select() if it's telling us the
socket is ready for data when it really isn't but oh well.

Use path not protocol as the path part of the uri of the folder_created

2001-10-28    <NotZed@Ximian.com>

* providers/local/camel-local-folder.c
(camel_local_folder_construct): Use path not protocol as the path
part of the uri of the folder_created event.

* providers/local/camel-maildir-store.c (scan_dir): If FAST is
set, dont try and scan for unread counts.

* providers/local/camel-local-store.c (create_folder): Implement,
just return the folderinfo of the folder matched, not all of it.

* camel-store.c (camel_store_rename_folder): Rename the
camelfolders before emitting the folderchanged event (otherwise
vstore fails to pick up change.

* camel-store.c (camel_store_init): Always use a default dir_sep
of '/'.

* camel-store.h (struct _CamelStore): Added a dir_sep to the base
store so rename and whatnot can work, temporary fix for mixed dir
separator in folder name api's.

* providers/imap/camel-map-store.c:
(imap_connect_online): Also set the store's dir_sep here.
(imap_connect_offline): "

More fixing of the license texts.

Fixing the license text.

Finished implementation.

2001-10-27    <NotZed@Ximian.com>

        * providers/imap/camel-imap-store.c (rename_folder): Finished
        implementation.

        * providers/local/camel-local-folder.c (local_rename): Implement
        local rename of folder objects.

2001-10-26    <NotZed@Ximian.com>

        * camel-vee-folder.c (camel_vee_folder_class_init): Hook into
        rename function.
        (vee_rename): Implement the veefolder rename function.
        (camel_vee_folder_class_init):
        (folder_changed_change): Kill a warning with a cast.

        * camel-vee-store.c (vee_rename_folder): Emit a folder_renamed
        event properly, also call parent to do some rename stuff.

        * camel-store.h: Added a CamelRenameInfo for the rename event.

        * camel-folder.c (camel_folder_rename): New function to rename a
        folder object.
        (camel_folder_class_init): Added a renamed event.
        (folder_rename): Default impl, set full_name, and set name
        assuming the dir separator is '/'.

        * camel-store.c (camel_store_class_init): Added folder_renamed
        event.
        (camel_store_rename_folder): Rename an active folder object if we
        have one, and update the folder table.
        (rename_folder): Make a default implementation that handles
        updating the folder tree.

Don't bother checking ex since we can rely on the response being NULL on

2001-10-26  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-store.c (delete_folder): Don't bother
checking ex since we can rely on the response being NULL on error.
(rename_folder): Implemented.

Update the licensing information to require version 2 of the GPL
(instead of version 2 or any later version).

Check to make sure that the dataset is non-NULL before using and/or

2001-10-26  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-folder.c (imap_get_message): Check to
make sure that the dataset is non-NULL before using and/or
freeing.

Check for EWOULDBLOCK too? (stream_read): Same. (stream_flush): Return 0

2001-10-26  Jeffrey Stedfast  <fejj@ximian.com>

* camel-tcp-stream-raw.c (stream_write): Check for EWOULDBLOCK
too?
(stream_read): Same.
(stream_flush): Return 0 always, fsync() and tcp sockets don't
play nice :-)

* providers/imap/camel-imap-message-cache.c
(camel_imap_message_cache_get): Lets try putting a
camel_stream_reset() here. Logic being that the stream may have
been read from since it was cached and thus our caller may try
reading from it and get no data from it since it is already at the
EOS. This may fix bug #12943.

Check for EWOULDBLOCK too? (stream_read): Same.

2001-10-26  Jeffrey Stedfast  <fejj@ximian.com>

* camel-tcp-stream-raw.c (stream_write): Check for EWOULDBLOCK too?
(stream_read): Same.

* providers/imap/camel-imap-message-cache.c
(camel_imap_message_cache_get): Lets try putting a
camel_stream_reset() here. Logic being that the stream may have
been read from since it was cached and thus our caller may try
reading from it and get no data from it since it is already at the
EOS. This may fix bug #12943.

Lets try putting a camel_stream_reset() here. Logic being that the stream

2001-10-26  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-message-cache.c
(camel_imap_message_cache_get): Lets try putting a
camel_stream_reset() here. Logic being that the stream may have
been read from since it was cached and thus our caller may try
reading from it and get no data from it since it is already at the
EOS. This may fix bug #12943.

Remove the stupid warning that should never have been there in the first

2001-10-25    <NotZed@Ximian.com>

        * camel-search-private.c (utf8_get): Remove the stupid warning
        that should never have been there in the first place.

        * camel-sasl-digest-md5.c (digest_response): s/iconv/e_iconv/

        * camel-pgp-context.c (pgp_verify): "

        * camel-mime-utils.c (rfc2047_decode_word, rfc2047_decode_word,
        append_8bit, rfc2047_encode_word, rfc2184_decode,
        header_decode_param): "

        * camel-mime-part-utils.c (convert_buffer, convert_buffer): "

        * camel-mime-filter-charset.c (reset, complete, filter): "

Don't let fi->name be NULL.

* providers/imap/camel-imap-store.c (unsubscribe_folder): Don't
let fi->name be NULL.

updated the SIMULATE_FLAKY_NETWORK code

Same as in the TcpStreamRaw code. (stream_write): And again here...

2001-10-24  Jeffrey Stedfast  <fejj@ximian.com>

* camel-stream-fs.c (stream_read): Same as in the TcpStreamRaw code.
(stream_write): And again here...

* camel-tcp-stream-raw.c (stream_read): Handle the EAGAIN error
case as well.
(stream_write): Same here, this might fix the SMTP truncation
thing? I hope?

Remove an extra * in one of the regexps that glibc apparently doesn't mind

* camel-mime-utils.c (mail_list_magic): Remove an extra * in one
of the regexps that glibc apparently doesn't mind but bsd does.

made d(x) x recompile again.

2001-10-23    <NotZed@Ximian.com>

        * providers/local/camel-mbox-summary.c: made d(x) x recompile again.

No, use Storing, as the other code does :p (spool_summary_check): Check

2001-10-22    <NotZed@Ximian.com>

        * providers/local/camel-spool-summary.c (summary_rebuild): No, use
        Storing, as the other code does :p
        (spool_summary_check): Check for consistency.

s/summarising/summarizing/. (spool_summary_check): s/summarise/summarize/.

2001-10-22  Jon Trowbridge  <trow@ximian.com>

* providers/local/camel-spool-summary.c (summary_rebuild):
s/summarising/summarizing/.
(spool_summary_check): s/summarise/summarize/.

If we are kludging around a inline-pgp signed part, do some charset

2001-10-22  Jeffrey Stedfast  <fejj@ximian.com>

* camel-pgp-mime.c (camel_pgp_mime_part_verify): If we are
kludging around a inline-pgp signed part, do some charset
conversion to protect any 8bit text.

Use search_type_mlist for mailing list searches.

* camel-filter-search.c, camel-folder-search.c (check_header): Use
search_type_mlist for mailing list searches.

* camel.c (camel_init): call camel-mime-utils-init func.

* camel-mime-utils.c: Changed mail mail_list_magic to include a
domain part, also pre-compile all the patterns.  They are all
backward compatible except List-Id: which now uses the
mail-address-like <list-name.host.name> part rather than the
plain-text part (which might've been blank anyway).
(camel_mime_utils_init): Initialisation function to setup any
static data required by camel-mime-utils at run-time.  We now
setup the base64/charset class tables here, so it doesn't need to
be done statically.
(camel_mime_special_table, camel_mime_base64_rank): No longer
statically initialise these.
(main): Removed + the tests at the end of the file.
(header_raw_check_mailing_list): Dont compile regex's here,
already compiled in utils_init.  Use the regex patterns to remove
leading <'s from addresses.  Also, if there is a domain part
matched, add that after a '@' is added.

camel-search-private.c:
(camel_search_header_match): Added SEARCH_TYPE_MLIST for mlist
search types.  It ignores the domain part if either (but not both)
of the values dont have it.

(camel_search_header_match): Oops, missing i=0.

Pass a CORBA_Environment to bonobo_config_get_string so it doesn't g_warn

* e-passwords.c (e_passwords_get_password): Pass a
CORBA_Environment to bonobo_config_get_string so it doesn't g_warn
on error. (Since the "error" is most likely just that the password
isn't cached.)
(e_passwords_remember_password, e_passwords_get_password,
e_passwords_add_password): Change "if (foo) { entire function; }"
to "if (!foo) return;"

* e-html-utils.c (special_chars): Don't allow single quote or
backtick in email addresses, or pipes following URLs.

Match check against all addresses in a multiple address match. Should fix

2001-10-22    <NotZed@Ximian.com>

        * camel-search-private.c (camel_search_header_match): Match check
        against all addresses in a multiple address match.  Should fix
        #13051.

Add a hack: if the multipart/signed part has a "x-inline-pgp-hack"

       * camel-pgp-mime.c (camel_pgp_mime_part_verify): Add a hack: if
       the multipart/signed part has a "x-inline-pgp-hack" parameter in
       its Content-Type, don't pass the MIME headers as part of the data
       to be verified.

added $GNOME_INCLUDEDIR to INCLUDES, needed for latest libgnome, which

2001-10-22  Rodrigo Moya <rodrigo@ximian.com>

* Makefile.am: added $GNOME_INCLUDEDIR to INCLUDES, needed for latest
libgnome, which install headers in $(prefix)/gnome-1.0

Only do the rawtext checks if the part is a text part, otherwise don't

2001-10-21  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-part-utils.c
(simple_data_wrapper_construct_from_parser): Only do the rawtext
checks if the part is a text part, otherwise don't bother wasting
cpu cycles.

Updated comments to make the code a bit more clear.

2001-10-21  Jeffrey Stedfast  <fejj@ximian.com>

* camel-store.c (camel_folder_info_build): Updated comments to
make the code a bit more clear.

Dont do the macro stuff - basically so you can actually debug calls.

2001-10-21    <NotZed@Ximian.com>

        * camel-store.c (CS_CLASS): Dont do the macro stuff - basically so
        you can actually debug calls.
        (camel_store_uri_cmp): New function to compare store objects.

        * camel-vee-folder.c (vee_folder_add_info): oops, dont free the
        vuid, since its alloca'd now.
        (folder_changed_change): Kill a compile warning.

oops, dont free the vuid, since its alloca'd now.

2001-10-21    <NotZed@Ximian.com>

* camel-vee-folder.c (vee_folder_add_info): oops, dont free the
vuid, since its alloca'd now.

If no exception supplied, use a local one. (move_message_to): Set the seen

2001-10-20    <NotZed@Ximian.com>

        * camel-folder.c (move_messages_to): If no exception supplied, use
        a local one.
        (move_message_to): Set the seen flag also when we delete it.

If the destination folder is the vfolder source folder, then

2001-10-19  Jeffrey Stedfast  <fejj@ximian.com>

* camel-vtrash-folder.c (vtrash_move_messages_to): If the
destination folder is the vfolder source folder, then
set_message_flags on it.

When registering provider, translate all strings.

2001-10-19    <NotZed@Ximian.com>

        * camel-session.c (register_provider): When registering provider,
        translate all strings.

camel-vee-folder.c (folder_changed_change): Change logic, we always add

camel-vee-folder.c
        (folder_changed_change): Change logic, we always add changed stuff
        if it now matches, but dont remove it unless its auto-remove, only
        propagate changes for it.
        (vee_set_message_flags): Call parent method after doing our work.
        (vee_set_message_user_flag): Same here.

Fixes a deadlock. camel-vee-folder.c: (folder_changed_change): Ok, so dont

Fixes a deadlock.
camel-vee-folder.c:
(folder_changed_change): Ok, so dont hold the subfolder lock for
the whole duration of the changed event, although we probably
should, requires a recursive mutex.

Typo, srlen aint right.

camel-vee-folder.c: (folder_changed_change_uid): Use alloca for vuid, and

camel-vee-folder.c:
(folder_changed_change_uid): Use alloca for vuid, and not
sprintf(".lengths")
(vee_search_by_expression): Dont use sprintf for vuid.
(vee_search_by_uids): "
(vee_folder_add_info): "
(folder_changed_remove_uid): "
(folder_changed_change): "
(folder_changed_change_uid): Also check folder_unmatched for
changed uid's so we properly update/propagate unmatched changes.

Lock the right lock for unmatched subfolder list stuff. If the sub folder

2001-10-19    <NotZed@Ximian.com>

* camel-vee-folder.c (camel_vee_folder_remove_folder): Lock the
right lock for unmatched subfolder list stuff.  If the sub folder
is deleted & part of unmatched, or the last ref to the sub folder
is gone from unmatched, blow it away completely from unmatched,
but only do it once.
(vee_folder_remove_folder): Added arg to kill unmatched
unconditionally.  Also handle deleted folders, always remove
stuff.
(folder_changed_change): Lock subfolder lock for entirety of
operation, and also check to see if the subfolder is still there
before doing any fancy work (incase it got removed while we were
waiting to execute).

* camel-store.c (camel_store_unsubscribe_folder): If we are
unsubscribing from a folder we have loaded, set the folder
deleted, as with the delete_folder case.

If a charset isn't specified or claims to be utf-8, check the validity of

2001-10-18  Jeffrey Stedfast  <fejj@ximian.com>

* camel-mime-part-utils.c
(simple_data_wrapper_construct_from_parser): If a charset isn't
specified or claims to be utf-8, check the validity of the text
and if it's invalid, set the rawtext bit to TRUE. If the charset
is x-unknown or some other x- charset, always set the rawtext bit
to TRUE.

Added CAMEL_CFLAGS so we get the right iconv stuff, patch from Yanko

2001-10-18    <NotZed@Ximian.com>

        * providers/(imap|local|pop3|sendmail|smtp)/Makefile.am: Added
        CAMEL_CFLAGS so we get the right iconv stuff, patch from Yanko
        Kaneti <yaneti@declera.com>

Make sure the cancel_fd isn't -1.

2001-10-18  Jeffrey Stedfast  <fejj@ximian.com>

* camel-pgp-context.c (crypto_exec_with_passwd): Make sure the
cancel_fd isn't -1.

Flush the stream here, not sure it really matters but it might (hopefully)

2001-10-18  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-message-cache.c (insert_finish): Flush
the stream here, not sure it really matters but it might
(hopefully) fix bug #12943.

Make user-cancellable.

2001-10-18  Jeffrey Stedfast  <fejj@ximian.com>

* camel-pgp-context.c (crypto_exec_with_passwd): Make
user-cancellable.

String cleanup for bug #6640.

2001-10-18  Jeffrey Stedfast  <fejj@ximian.com>

* providers/local/camel-local-provider.c: String cleanup for bug
#6640.

Fix my Cyrus imapd workaround...instead of checking to see if info->flags

2001-10-18  Jeffrey Stedfast  <fejj@ximian.com>

* providers/imap/camel-imap-folder.c (imap_sync_online): Fix my
Cyrus imapd workaround...instead of checking to see if info->flags
== FLAGGED, just check to see if it contains any IMAP flags.

Use '.' as the inbox name for filter new messages test.

2001-10-17    <NotZed@Ximian.com>

        * providers/local/camel-maildir-folder.c
        (camel_maildir_folder_new): Use '.' as the inbox name for filter
        new messages test.

New function to clone a folderinfo tree.

2001-10-17    <NotZed@Ximian.com>

        * camel-store.c (camel_folder_info_clone): New function to clone a
        folderinfo tree.

Same as maildir and mbox append-message.

2001-10-17  Jeffrey Stedfast  <fejj@ximian.com>

* providers/local/camel-mh-folder.c (mh_append_message): Same as
maildir and mbox append-message.

* providers/local/camel-maildir-folder.c (maildir_append_message):
Same as mbox append.

* camel-exception.c (camel_exception_set): Make sure the new
description isn't the same as the old description pointer before
freeing the old one.
(camel_exception_setv): Don't free the old description until we
set the new one this way we can reuse the old description in the
new description.

* providers/local/camel-mbox-folder.c (mbox_append_message): If
errno == EINTR, then we got a user-cancel so set the exception
appropriately so that we don't make the user shit his pants.