protocols/imap/src/main/java/org/apache/james/imap/decode/parser/UidCommandParser.java (6 lines):
	- line 53: // TODO: check the logic against the specification:
	- line 54: // TODO: suspect that it is now bust
	- line 55: // TODO: the command written may be wrong
	- line 56: // TODO: this will be easier to fix a little later
	- line 57: // TODO: also not sure whether the old implementation shares this flaw
	- line 60: // TODO: replace abstract class with interface


protocols/imap/src/main/java/org/apache/james/imap/decode/ImapRequestLineReader.java (5 lines):
	- line 59: * lookahead=1 on the underlying character stream. TODO need to look at encoding
	- line 193: // TODO: nextChar does not report accurate chars so safe to
	- line 287: * @return The next character TODO: character encoding is variable and
	- line 297: * filling the char array provided. TODO: remove unnecessary copying of
	- line 686: * Consumes a CRLF from the request. TODO: This is too liberal, the spec


protocols/imap/src/main/java/org/apache/james/imap/encode/base/ImapResponseComposerImpl.java (3 lines):
	- line 121: // TODO: consider message normalisation
	- line 122: // TODO: CR/NFs in message must be replaced
	- line 123: // TODO: probably best done in the writer


server/dns-service/dnsservice-dnsjava/src/main/java/org/apache/james/dnsservice/dnsjava/DNSJavaService.java (3 lines):
	- line 187: // TODO: Check to see if the credibility field is being used correctly.
	- line 386: // TODO find out if it's a bug in DNSJava
	- line 393: // TODO: Figure out how to handle this correctly.


backends-common/opensearch/src/main/java/org/apache/james/backends/opensearch/json/JsonpUtils.java (3 lines):
	- line 131: // FIXME: need a buffering parser wrapper so that we don't roundtrip through a JsonObject and a String
	- line 132: // FIXME: resulting parser should return locations that are offset with the original parser's location
	- line 151: // FIXME: we should have used createParser(object), but this doesn't work as it creates a


mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMailboxManager.java (2 lines):
	- line 385: // TODO: transaction
	- line 640: // TODO put this into a serilizable transaction


protocols/imap/src/main/java/org/apache/james/imap/encode/FetchResponseEncoder.java (2 lines):
	- line 290: // TODO: add method to composer
	- line 299: // TODO: add method to composer


backends-common/opensearch/src/main/java/org/apache/james/backends/opensearch/json/jackson/JacksonJsonProvider.java (2 lines):
	- line 82: // TODO: handle specific configuration
	- line 167: // TODO: handle specific configuration


server/protocols/protocols-smtp/src/main/java/org/apache/james/smtpserver/DataLineJamesMessageHookHandler.java (2 lines):
	- line 108: // TODO probably return a temporary problem
	- line 122: // TODO: maybe we should handle the Header/Body recognition here


protocols/imap/src/main/java/org/apache/james/imap/processor/ListRightsProcessor.java (2 lines):
	- line 103: // FIXME check if identifier is a valid user or group
	- line 104: // FIXME Servers, when processing a command that has an identifier as a


mailbox/store/src/main/java/org/apache/james/mailbox/store/StoreMessageManager.java (2 lines):
	- line 733: //TODO lock the from mailbox too, in a non-deadlocking manner - how?
	- line 750: //TODO lock the from mailbox too, in a non-deadlocking manner - how?


server/mailet/mailetcontainer-impl/src/main/java/org/apache/james/mailetcontainer/impl/JamesMailetContext.java (2 lines):
	- line 110: // TODO: We only do this to not break backward compatiblity. Should
	- line 286: // TODO: We only do this to not break backward compatiblity. Should


protocols/imap/src/main/java/org/apache/james/imap/encode/ImapEncoder.java (2 lines):
	- line 27: * TODO: correct this API
	- line 32: * Writes response. TODO: pass in writer rather than composer


protocols/netty/src/main/java/org/apache/james/protocols/netty/BasicChannelInboundHandler.java (2 lines):
	- line 113: // TODO: This kind of sucks but I was able to come up with something more elegant here
	- line 191: // TODO: This kind of sucks but I was able to come up with something more elegant here


protocols/smtp/src/main/java/org/apache/james/protocols/smtp/core/esmtp/AuthCmdHandler.java (2 lines):
	- line 388: // TODO: Make this string a more useful debug message
	- line 552: // TODO we should at least check for correct syntax and put the result in session


protocols/imap/src/main/java/org/apache/james/imap/encode/main/DefaultLocalizer.java (2 lines):
	- line 42: //FIXME implement the locale selection
	- line 44: //FIXME implement the localized value lookup depending on chosenLocale


protocols/imap/src/main/java/org/apache/james/imap/main/DefaultImapDecoderFactory.java (1 line):
	- line 30: * TODO: this is temporary: should let the container do the coupling. TODO:


mailbox/jpa/src/main/java/org/apache/james/mailbox/jpa/mail/model/openjpa/JPAStreamingMailboxMessage.java (1 line):
	- line 50: * TODO: Fix me!


server/protocols/fetchmail/src/main/java/org/apache/james/fetchmail/ProcessorAbstract.java (1 line):
	- line 97: // TODO Consider making this configurable


protocols/imap/src/main/java/org/apache/james/imap/encode/StatusResponseEncoder.java (1 line):
	- line 109: // TODO: calculate locales


protocols/smtp/src/main/java/org/apache/james/protocols/smtp/core/DataLineMessageHookHandler.java (1 line):
	- line 82: // TODO: maybe we should handle the Header/Body recognition here


protocols/smtp/src/main/java/org/apache/james/protocols/smtp/MailEnvelope.java (1 line):
	- line 75: * TODO: Think about how to remove this!


protocols/imap/src/main/java/org/apache/james/imap/encode/ContinuationResponseEncoder.java (1 line):
	- line 52: // TODO: calculate locales


server/dns-service/dnsservice-test/src/main/java/org/apache/james/dnsservice/api/AbstractDNSServiceTest.java (1 line):
	- line 85: //TODO: move to some sort of Live tests


protocols/imap/src/main/java/org/apache/james/imap/decode/ImapCommandParser.java (1 line):
	- line 36: * Parses IMAP request. TODO: consider error handling


protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java (1 line):
	- line 305: // TODO: consider whether error message should be passed to the user


server/container/core/src/main/java/org/apache/james/server/core/MimeMessageWrapper.java (1 line):
	- line 226: // TODO is it ok?


protocols/imap/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java (1 line):
	- line 53: // TODO: this should be configurable


server/data/data-library/src/main/java/org/apache/james/repository/file/FilePersistentStreamRepository.java (1 line):
	- line 34: * TODO: -retieve(String key) should return a FilterInputStream to allow


protocols/imap/src/main/java/org/apache/james/imap/processor/fetch/MimeDescriptorStructure.java (1 line):
	- line 81: // TODO: consider revising this


protocols/imap/src/main/java/org/apache/james/imap/processor/fetch/FetchProcessor.java (1 line):
	- line 137: // TODO: From the QRESYNC RFC it seems ok to send the VANISHED responses after the FETCH Responses.


server/container/core/src/main/java/org/apache/james/server/core/MailImpl.java (1 line):
	- line 558: // TODO: These need some work. Currently very tightly coupled to


mailet/standard/src/main/java/org/apache/james/transport/mailets/WrapText.java (1 line):
	- line 56: // TODO We could even manage the flow when the message is quoted-printable


mailbox/backup/src/main/java/org/apache/james/mailbox/backup/ZipMailArchiveRestorer.java (1 line):
	- line 87: //Ignore for know, TODO: implementation


server/protocols/protocols-smtp/src/main/java/org/apache/james/smtpserver/fastfail/ValidSenderDomainHandler.java (1 line):
	- line 49: // TODO: Should we reject temporary ?


server/protocols/protocols-imap4/src/main/java/org/apache/james/imapserver/netty/NettyStreamImapRequestLineReader.java (1 line):
	- line 101: * @return The next character TODO: character encoding is variable and


server/queue/queue-activemq/src/main/java/org/apache/james/queue/activemq/ActiveMQCacheableMailQueue.java (1 line):
	- line 178: * TODO: Re-Enable it again once it works!


mailet/crypto/src/main/java/org/apache/james/transport/matcher/IsX509CertificateSubject.java (1 line):
	- line 84: // TODO: Maybe here a more strong check should be done ...


server/dns-service/dnsservice-library/src/main/java/org/apache/james/dnsservice/library/inetnetwork/model/Inet6Network.java (1 line):
	- line 127: // TODO Don't know if this is correct?


protocols/imap/src/main/java/org/apache/james/imap/message/response/FetchResponse.java (1 line):
	- line 148: * TODO: replace


protocols/imap/src/main/java/org/apache/james/imap/api/ImapConstants.java (1 line):
	- line 121: // TODO: Consider switching to standard case


server/data/data-jpa/src/main/java/org/apache/james/mailrepository/jpa/model/JPAMail.java (1 line):
	- line 121: private byte[] messageBody; // TODO: support streaming body where possible (see e.g. JPAStreamingMailboxMessage)


mailbox/tools/jpa-migrator/src/main/java/org/apache/james/mailbox/tools/jpa/migrator/JpaMigrator.java (1 line):
	- line 50: * TODO Extract the SQL in JAVA classes to XML file.


mailbox/lucene/src/main/java/org/apache/james/mailbox/lucene/search/LuceneMessageSearchIndex.java (1 line):
	- line 566: // TODO: Better handling


mailbox/store/src/main/java/org/apache/james/mailbox/store/MessageResultImpl.java (1 line):
	- line 308: * TODO: Move to Mime4j


protocols/imap/src/main/java/org/apache/james/imap/decode/ImapRequestStreamLineReader.java (1 line):
	- line 51: * @return The next character TODO: character encoding is variable and


mailet/standard/src/main/java/org/apache/james/transport/mailets/UnwrapText.java (1 line):
	- line 75: // TODO replace non standard quotes (at least "> " with ">", otherwise the widely used  "> > >" will not work.


mailet/crypto/src/main/java/org/apache/james/transport/package.html (1 line):
	- line 21: <p>TODO: Should probably rearrange class hierarchy and eliminate this package.</p>


protocols/imap/src/main/java/org/apache/james/imap/encode/main/DefaultImapEncoderFactory.java (1 line):
	- line 63: * TODO: perhaps a POJO would be better


server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/SPF.java (1 line):
	- line 121: // TODO use once fixed AsynchronousSPFExecutor (see JAMES-3920)


mpt/core/src/main/java/org/apache/james/mpt/protocol/ProtocolSession.java (1 line):
	- line 41: * the server response against the expected values. TODO make ProtocolSession


server/apps/spring-app/src/main/resources/smtpserver.xml (1 line):
	- line 142: <!-- TODO: CHANGE TO OFFICIAL URL LATER -->


server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/remote/delivery/HeloNameProvider.java (1 line):
	- line 42: // TODO: Maybe we should better just lookup the hostname via dns


protocols/netty/src/main/java/org/apache/james/protocols/netty/LineHandlerUpstreamHandler.java (1 line):
	- line 54: // TODO: This kind of sucks but I was not able to come up with something more elegant here


server/mailet/mailets/src/main/java/org/apache/james/transport/mailets/jsieve/delivery/SievePoster.java (1 line):
	- line 70: // TODO: When user missing, append to a default location