projects
/
apps
/
madmutt.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
remove most of the debug code: often makes the code unreadable, for little
[apps/madmutt.git]
/
pop
/
pop.c
diff --git
a/pop/pop.c
b/pop/pop.c
index
465a770
..
5fe55e4
100644
(file)
--- a/
pop/pop.c
+++ b/
pop/pop.c
@@
-11,20
+11,20
@@
# include "config.h"
#endif
# include "config.h"
#endif
+#include <string.h>
+#include <unistd.h>
+
#include <lib-lib/mem.h>
#include <lib-lib/mem.h>
+#include <lib-lib/file.h>
+#include <lib-lib/str.h>
+#include <lib-lib/macros.h>
+
+#include <lib-ui/curses.h>
#include "mutt.h"
#include "mx.h"
#include "pop.h"
#include "mutt.h"
#include "mx.h"
#include "pop.h"
-#include "mutt_crypt.h"
-#include "mutt_curses.h"
-
-#include "lib/str.h"
-#include "lib/intl.h"
-#include "lib/debug.h"
-
-#include <string.h>
-#include <unistd.h>
+#include <lib-crypt/crypt.h>
/* write line to file */
static int fetch_message (char *line, void *file)
/* write line to file */
static int fetch_message (char *line, void *file)
@@
-49,7
+49,7
@@
static int fetch_message (char *line, void *file)
static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h)
{
FILE *f;
static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h)
{
FILE *f;
- int i
nde
x;
+ int i
d
x;
pop_query_status ret;
long length;
char buf[LONG_STRING];
pop_query_status ret;
long length;
char buf[LONG_STRING];
@@
-61,10
+61,10
@@
static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h)
return PFD_FUNCT_ERROR;
}
return PFD_FUNCT_ERROR;
}
- snprintf (buf, sizeof (buf), "
LIST
%d\r\n", h->refno);
+ snprintf (buf, sizeof (buf), "
string_list_t
%d\r\n", h->refno);
ret = pop_query (pop_data, buf, sizeof (buf));
if (ret == PQ_OK) {
ret = pop_query (pop_data, buf, sizeof (buf));
if (ret == PQ_OK) {
- sscanf (buf, "+OK %d %ld", &i
nde
x, &length);
+ sscanf (buf, "+OK %d %ld", &i
d
x, &length);
snprintf (buf, sizeof (buf), "TOP %d 0\r\n", h->refno);
ret = pop_fetch_data (pop_data, buf, NULL, fetch_message, f);
snprintf (buf, sizeof (buf), "TOP %d 0\r\n", h->refno);
ret = pop_fetch_data (pop_data, buf, NULL, fetch_message, f);
@@
-72,14
+72,10
@@
static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h)
if (pop_data->cmd_top == CMD_UNKNOWN) {
if (ret == PQ_OK) {
pop_data->cmd_top = CMD_AVAILABLE;
if (pop_data->cmd_top == CMD_UNKNOWN) {
if (ret == PQ_OK) {
pop_data->cmd_top = CMD_AVAILABLE;
-
- debug_print (1, ("set TOP capability\n"));
}
if (ret == PQ_ERR) {
pop_data->cmd_top = CMD_NOT_AVAILABLE;
}
if (ret == PQ_ERR) {
pop_data->cmd_top = CMD_NOT_AVAILABLE;
-
- debug_print (1, ("unset TOP capability\n"));
snprintf (pop_data->err_msg, sizeof (pop_data->err_msg),
_("Command TOP is not supported by server."));
}
snprintf (pop_data->err_msg, sizeof (pop_data->err_msg),
_("Command TOP is not supported by server."));
}
@@
-125,30
+121,28
@@
static pop_query_status pop_read_header (POP_DATA * pop_data, HEADER * h)
/* parse UIDL */
static int fetch_uidl (char *line, void *data)
{
/* parse UIDL */
static int fetch_uidl (char *line, void *data)
{
- int i, i
nde
x;
+ int i, i
d
x;
CONTEXT *ctx = (CONTEXT *) data;
POP_DATA *pop_data = (POP_DATA *) ctx->data;
CONTEXT *ctx = (CONTEXT *) data;
POP_DATA *pop_data = (POP_DATA *) ctx->data;
- sscanf (line, "%d %s", &i
nde
x, line);
+ sscanf (line, "%d %s", &i
d
x, line);
for (i = 0; i < ctx->msgcount; i++)
for (i = 0; i < ctx->msgcount; i++)
- if (!
str_cmp
(line, ctx->hdrs[i]->data))
+ if (!
m_strcmp
(line, ctx->hdrs[i]->data))
break;
if (i == ctx->msgcount) {
break;
if (i == ctx->msgcount) {
- debug_print (1, ("new header %d %s\n", index, line));
-
if (i >= ctx->hdrmax)
mx_alloc_memory (ctx);
ctx->msgcount++;
if (i >= ctx->hdrmax)
mx_alloc_memory (ctx);
ctx->msgcount++;
- ctx->hdrs[i] =
mutt_new_header
();
- ctx->hdrs[i]->data =
str_dup
(line);
+ ctx->hdrs[i] =
header_new
();
+ ctx->hdrs[i]->data =
m_strdup
(line);
}
}
- else if (ctx->hdrs[i]->index != i
nde
x - 1)
+ else if (ctx->hdrs[i]->index != i
d
x - 1)
pop_data->clear_cache = 1;
pop_data->clear_cache = 1;
- ctx->hdrs[i]->refno = i
nde
x;
- ctx->hdrs[i]->index = i
nde
x - 1;
+ ctx->hdrs[i]->refno = i
d
x;
+ ctx->hdrs[i]->index = i
d
x - 1;
return 0;
}
return 0;
}
@@
-181,14
+175,11
@@
static int pop_fetch_headers (CONTEXT * ctx)
if (pop_data->cmd_uidl == CMD_UNKNOWN) {
if (ret == PQ_OK) {
pop_data->cmd_uidl = CMD_AVAILABLE;
if (pop_data->cmd_uidl == CMD_UNKNOWN) {
if (ret == PQ_OK) {
pop_data->cmd_uidl = CMD_AVAILABLE;
-
- debug_print (1, ("set UIDL capability\n"));
}
if (ret == PQ_ERR && pop_data->cmd_uidl == CMD_UNKNOWN) {
pop_data->cmd_uidl = CMD_NOT_AVAILABLE;
}
if (ret == PQ_ERR && pop_data->cmd_uidl == CMD_UNKNOWN) {
pop_data->cmd_uidl = CMD_NOT_AVAILABLE;
- debug_print (1, ("unset UIDL capability\n"));
snprintf (pop_data->err_msg, sizeof (pop_data->err_msg),
_("Command UIDL is not supported by server."));
}
snprintf (pop_data->err_msg, sizeof (pop_data->err_msg),
_("Command UIDL is not supported by server."));
}
@@
-216,7
+207,7
@@
static int pop_fetch_headers (CONTEXT * ctx)
if (ret != PQ_OK) {
for (i = ctx->msgcount; i < new_count; i++)
if (ret != PQ_OK) {
for (i = ctx->msgcount; i < new_count; i++)
-
mutt_free_header
(&ctx->hdrs[i]);
+
header_delete
(&ctx->hdrs[i]);
return ret;
}
return ret;
}
@@
-230,25
+221,25
@@
int pop_open_mailbox (CONTEXT * ctx)
int ret;
char buf[LONG_STRING];
CONNECTION *conn;
int ret;
char buf[LONG_STRING];
CONNECTION *conn;
- ACCOUNT ac
c
t;
+ ACCOUNT act;
POP_DATA *pop_data;
ciss_url_t url;
POP_DATA *pop_data;
ciss_url_t url;
- if (pop_parse_path (ctx->path, &ac
c
t)) {
+ if (pop_parse_path (ctx->path, &act)) {
mutt_error (_("%s is an invalid POP path"), ctx->path);
mutt_sleep (2);
return -1;
}
mutt_error (_("%s is an invalid POP path"), ctx->path);
mutt_sleep (2);
return -1;
}
- mutt_account_tourl (&ac
c
t, &url);
+ mutt_account_tourl (&act, &url);
url.path = NULL;
url_ciss_tostring (&url, buf, sizeof (buf), 0);
url.path = NULL;
url_ciss_tostring (&url, buf, sizeof (buf), 0);
- conn = mutt_conn_find (NULL, &ac
c
t);
+ conn = mutt_conn_find (NULL, &act);
if (!conn)
return -1;
p_delete(&ctx->path);
if (!conn)
return -1;
p_delete(&ctx->path);
- ctx->path =
str_dup
(buf);
+ ctx->path =
m_strdup
(buf);
pop_data = p_new(POP_DATA, 1);
pop_data->conn = conn;
pop_data = p_new(POP_DATA, 1);
pop_data->conn = conn;
@@
-259,7
+250,7
@@
int pop_open_mailbox (CONTEXT * ctx)
conn->data = pop_data;
conn->data = pop_data;
-
FOREVER
{
+
for (;;)
{
if (pop_reconnect (ctx) != PQ_OK)
return -1;
if (pop_reconnect (ctx) != PQ_OK)
return -1;
@@
-287,8
+278,6
@@
static void pop_clear_cache (POP_DATA * pop_data)
if (!pop_data->clear_cache)
return;
if (!pop_data->clear_cache)
return;
- debug_print (1, ("delete cached messages\n"));
-
for (i = 0; i < POP_CACHE_LEN; i++) {
if (pop_data->cache[i].path) {
unlink (pop_data->cache[i].path);
for (i = 0; i < POP_CACHE_LEN; i++) {
if (pop_data->cache[i].path) {
unlink (pop_data->cache[i].path);
@@
-354,7
+343,7
@@
int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno)
}
}
}
}
-
FOREVER
{
+
for (;;)
{
if (pop_reconnect (ctx) != PQ_OK)
return -1;
if (pop_reconnect (ctx) != PQ_OK)
return -1;
@@
-405,10
+394,10
@@
int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno)
* portion of the headers, those required for the main display.
*/
cache->index = h->index;
* portion of the headers, those required for the main display.
*/
cache->index = h->index;
- cache->path =
str_dup
(path);
+ cache->path =
m_strdup
(path);
rewind (msg->fp);
uidl = h->data;
rewind (msg->fp);
uidl = h->data;
-
mutt_free_envelope
(&h->env);
+
envelope_delete
(&h->env);
h->env = mutt_read_rfc822_header (msg->fp, h, 0, 0);
h->data = uidl;
h->lines = 0;
h->env = mutt_read_rfc822_header (msg->fp, h, 0, 0);
h->data = uidl;
h->lines = 0;
@@
-421,8
+410,7
@@
int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno)
h->content->length = ftello (msg->fp) - h->content->offset;
/* This needs to be done in case this is a multipart message */
h->content->length = ftello (msg->fp) - h->content->offset;
/* This needs to be done in case this is a multipart message */
- if (!WithCrypto)
- h->security = crypt_query (h->content);
+ h->security = crypt_query (h->content);
mutt_clear_error ();
rewind (msg->fp);
mutt_clear_error ();
rewind (msg->fp);
@@
-431,7
+419,9
@@
int pop_fetch_message (MESSAGE * msg, CONTEXT * ctx, int msgno)
}
/* update POP mailbox - delete messages from server */
}
/* update POP mailbox - delete messages from server */
-pop_query_status pop_sync_mailbox (CONTEXT * ctx, int unused, int *index_hint)
+pop_query_status pop_sync_mailbox (CONTEXT * ctx,
+ int unused __attribute__ ((unused)),
+ int *index_hint __attribute__ ((unused)))
{
int i;
pop_query_status ret;
{
int i;
pop_query_status ret;
@@
-440,7
+430,7
@@
pop_query_status pop_sync_mailbox (CONTEXT * ctx, int unused, int *index_hint)
pop_data->check_time = 0;
pop_data->check_time = 0;
-
FOREVER
{
+
for (;;)
{
if (pop_reconnect (ctx) != PQ_OK)
return PQ_NOT_CONNECTED;
if (pop_reconnect (ctx) != PQ_OK)
return PQ_NOT_CONNECTED;
@@
-454,7
+444,7
@@
pop_query_status pop_sync_mailbox (CONTEXT * ctx, int unused, int *index_hint)
}
if (ret == PQ_OK) {
}
if (ret == PQ_OK) {
-
strfcpy (buf, "QUIT\r\n", sizeof (buf)
);
+
m_strcpy(buf, sizeof(buf), "QUIT\r\n"
);
ret = pop_query (pop_data, buf, sizeof (buf));
}
ret = pop_query (pop_data, buf, sizeof (buf));
}
@@
-474,7
+464,9
@@
pop_query_status pop_sync_mailbox (CONTEXT * ctx, int unused, int *index_hint)
}
/* Check for new messages and fetch headers */
}
/* Check for new messages and fetch headers */
-int pop_check_mailbox (CONTEXT * ctx, int *index_hint, int unused)
+int pop_check_mailbox (CONTEXT * ctx,
+ int *index_hint __attribute__ ((unused)),
+ int unused __attribute__ ((unused)))
{
int ret;
POP_DATA *pop_data = (POP_DATA *) ctx->data;
{
int ret;
POP_DATA *pop_data = (POP_DATA *) ctx->data;
@@
-516,7
+508,7
@@
void pop_fetch_mail (void)
CONNECTION *conn;
CONTEXT ctx;
MESSAGE *msg = NULL;
CONNECTION *conn;
CONTEXT ctx;
MESSAGE *msg = NULL;
- ACCOUNT ac
c
t;
+ ACCOUNT act;
POP_DATA *pop_data;
if (!PopHost) {
POP_DATA *pop_data;
if (!PopHost) {
@@
-532,14
+524,14
@@
void pop_fetch_mail (void)
}
strcpy (p, PopHost); /* __STRCPY_CHECKED__ */
}
strcpy (p, PopHost); /* __STRCPY_CHECKED__ */
- ret = pop_parse_path (url, &ac
c
t);
+ ret = pop_parse_path (url, &act);
p_delete(&url);
if (ret) {
mutt_error (_("%s is an invalid POP path"), PopHost);
return;
}
p_delete(&url);
if (ret) {
mutt_error (_("%s is an invalid POP path"), PopHost);
return;
}
- conn = mutt_conn_find (NULL, &ac
c
t);
+ conn = mutt_conn_find (NULL, &act);
if (!conn)
return;
if (!conn)
return;
@@
-557,7
+549,7
@@
void pop_fetch_mail (void)
mutt_message _("Checking for new messages...");
/* find out how many messages are in the mailbox. */
mutt_message _("Checking for new messages...");
/* find out how many messages are in the mailbox. */
-
strfcpy (buffer, "STAT\r\n", sizeof (buffer)
);
+
m_strcpy(buffer, sizeof(buffer), "STAT\r\n"
);
ret = pop_query (pop_data, buffer, sizeof (buffer));
if (ret == PQ_NOT_CONNECTED)
goto fail;
ret = pop_query (pop_data, buffer, sizeof (buffer));
if (ret == PQ_NOT_CONNECTED)
goto fail;
@@
-570,7
+562,7
@@
void pop_fetch_mail (void)
/* only get unread messages */
if (msgs > 0 && option (OPTPOPLAST)) {
/* only get unread messages */
if (msgs > 0 && option (OPTPOPLAST)) {
-
strfcpy (buffer, "LAST\r\n", sizeof (buffer)
);
+
m_strcpy(buffer, sizeof(buffer), "LAST\r\n"
);
ret = pop_query (pop_data, buffer, sizeof (buffer));
if (ret == PQ_NOT_CONNECTED)
goto fail;
ret = pop_query (pop_data, buffer, sizeof (buffer));
if (ret == PQ_NOT_CONNECTED)
goto fail;
@@
-639,14
+631,14
@@
void pop_fetch_mail (void)
if (rset) {
/* make sure no messages get deleted */
if (rset) {
/* make sure no messages get deleted */
-
strfcpy (buffer, "RSET\r\n", sizeof (buffer)
);
+
m_strcpy(buffer, sizeof(buffer), "RSET\r\n"
);
if (pop_query (pop_data, buffer, sizeof (buffer)) == PQ_NOT_CONNECTED)
goto fail;
}
finish:
/* exit gracefully */
if (pop_query (pop_data, buffer, sizeof (buffer)) == PQ_NOT_CONNECTED)
goto fail;
}
finish:
/* exit gracefully */
-
strfcpy (buffer, "QUIT\r\n", sizeof (buffer)
);
+
m_strcpy(buffer, sizeof(buffer), "QUIT\r\n"
);
if (pop_query (pop_data, buffer, sizeof (buffer)) == PQ_NOT_CONNECTED)
goto fail;
mutt_socket_close (conn);
if (pop_query (pop_data, buffer, sizeof (buffer)) == PQ_NOT_CONNECTED)
goto fail;
mutt_socket_close (conn);