etexteditor / e Goto Github PK

In Ubuntu, when you invoke 'find in project', and click on a link in the search results, e crashes.

Version: 1.0.34b
Platform: Ubuntu Jaunty

The codebase has lots of commented-out code to handle a toolbar. It's very likely that new work on the codebase will take this toolbar code further and further from being in a working state.

(A) Is it OK to start removing this dead code or (B) should it be considered a work-in-progress and re-enabled?

I cannot build e from the master branch (on Linux).

Could you branch or tag a buildable one?

The "Filter through command" dialog should remember it's previous setting. I often use this feature and it's a pain that it has no memory of what I just did.

To reproduce:
Ctrl-H (Filter Through Command)
Enter the following:
Command: wc
Input: Document
Output: Tooltip
click Execute
Tooltip works as expected
Ctrl-H again should default to previous settings, but doesn't

Build environment is up-to-date and clean, built e using the Debug settings on a Vista Ultimate x64 machine.

How to reproduce the rename file-delete bug:

1. Click on a file once, wait a while, click on it again to get into rename-mode
2. BEFORE e triggers renaming, press delete and delete the file
3. Upon finish renaming, click on another file
4. e crashes without callstack

Also, since the big merge the other day, double clicking on folders in the project pane doesn't work. (I can file another bug for this one if you want me to)

I am getting errors trying to run build_externals-linux.sh. I am running Ubuntu Jaunty x64. The error seems to be in auibook.cpp. Would appreciate help!

./src/aui/auibook.cpp: In constructor ‘wxAuiSimpleTabArt::wxAuiSimpleTabArt()’
./src/aui/auibook.cpp:1002: error: ‘m_pressed_close_bmp’ was not declared in this scope
./src/aui/auibook.cpp:1003: error: ‘m_hot_close_bmp’ was not declared in this scope
./src/aui/auibook.cpp: At global scope:
./src/aui/auibook.cpp:4799: error: redefinition of ‘int wxAuiNotebook::PageToTab(size_t)’
./src/aui/auibook.cpp:4617: error: ‘int wxAuiNotebook::PageToTab(size_t)’ previously defined here
./src/aui/auibook.cpp:4827: error: redefinition of ‘int wxAuiNotebook::TabToPage(size_t)’
./src/aui/auibook.cpp:4645: error: ‘int wxAuiNotebook::TabToPage(size_t)’ previously defined here
./src/aui/auibook.cpp:4851: error: redefinition of ‘wxString wxAuiNotebook::SavePerspective()’
./src/aui/auibook.cpp:4669: error: ‘wxString wxAuiNotebook::SavePerspective()’ previously defined here
./src/aui/auibook.cpp:4891: error: redefinition of ‘bool wxAuiNotebook::LoadPerspective(const wxString&)’
./src/aui/auibook.cpp:4709: error: ‘bool wxAuiNotebook::LoadPerspective(const wxString&)’ previously defined here
make: *** [monolib_auibook.o] Error 1
Cannot compile wxWidgets
~/etexteditor-e-ef9650485a4a69b715780ead1fc0dcc7b4cf4438/external
~/etexteditor-e-ef9650485a4a69b715780ead1fc0dcc7b4cf4438/external/webkit ~/etexteditor-e-ef9650485a4a69b715780ead1fc0dcc7b4cf4438/external
make[1]: Entering directory /home/ramin/etexteditor-e-ef9650485a4a69b715780ead1fc0dcc7b4cf4438/external/webkit/JavaScriptCore' ( xcodebuild -target All -alltargets cleanperl -I../WebKitTools/Scripts -Mwebkitdirs -e 'print XcodeOptionString()'| grep -v setenv && exit ${PIPESTATUS[0]} ) /bin/sh: xcodebuild: not found make[1]: *** [clean] Error 1 make[1]: Leaving directory/home/ramin/etexteditor-e-ef9650485a4a69b715780ead1fc0dcc7b4cf4438/external/webkit/JavaScriptCore'
make: *** [clean] Error 2

It would make things much more simple IMO. I can give it try, though I'm concerned about all the patches being applied to external libraries

./configure
make
make install

I found a few additional packages that are needed in order to get this to compile under Jaunty (9.04) after following the linux-notes.

libcurl4-dev (yes, I know it says you need libcurl-dev ... but that package doesn't seem to exist in jaunty)
libxml2-dev
gtk-doc-tools (this might be a meta package & have too much other stuff... but I couldn't find a specific package with "gtkdocize" in a shorter period of time)
libsoup2.4-dev
libgstreamer0.10-dev
libgstreamer-plugins-base0.10-dev

most of these might just be dependents used by the wx stuff... so they may not directly relevant to E's source... but I couldn't get it to compile without them.

On ubuntu, you can give E a command-line parameter, to cause E to open a certain file or project directory.

This only works if E is not running. If E is already started, invoking E a second time from the command line fails on Ubuntu. (it works on Windows)

Version: 1.0.34b
Platform: Ubuntu jaunty

EditorCtrls get passed a CatalystWrapper& reference, which they store in a reference member variable.

RevTooltip gets passed a CatalystWrapper (not a reference), which is stored in a non-reference member variable.

Is there a checklist for when to use what?

On first look it seems that changing RevTooltip to also use references wouldn't be a bad thing, but I wonder if they are using non-references due to a big fix.

Hi.

When compiling I still had this error message :
$ make DEBUG=1
[CC] tmKey.cpp
tmKey.cpp: In static member function ‘static void tmKey::BuildMaps()’:
tmKey.cpp:162: error: ‘’ was not declared in this scope
tmKey.cpp: In constructor ‘tmKey::tmKey(const wxString&)’:
tmKey.cpp:277: error: ‘wxMessageBox’ was not declared in this scope
tmKey.cpp: In member function ‘void tmKey::UpdateShortcut()’:
tmKey.cpp:291: error: ‘’ was not declared in this scope
tmKey.cpp:292: error: ‘’ was not declared in this scope
tmKey.cpp:293: error: ‘’ was not declared in this scope
tmKey.cpp:294: error: ‘’ was not declared in this scope
tmKey.cpp:300: error: ‘’ was not declared in this scope
tmKey.cpp: In member function ‘wxString tmKey::getBinding()’:
tmKey.cpp:321: error: ‘’ was not declared in this scope
tmKey.cpp:322: error: ‘’ was not declared in this scope
tmKey.cpp:323: error: ‘_’ was not declared in this scope
tmKey.cpp: In static member function ‘static bool tmKey::wxkToUni(int, bool, wxChar&)’:
tmKey.cpp:374: error: ‘gdk_unicode_to_keyval’ was not declared in this scope
tmKey.cpp:377: error: ‘gdk_keyval_to_upper’ was not declared in this scope
tmKey.cpp:377: error: ‘gdk_keyval_to_lower’ was not declared in this scope
tmKey.cpp:378: error: ‘gdk_keyval_to_unicode’ was not declared in this scope
tmKey.cpp:381: error: ‘wxLogDebug’ was not declared in this scope
tmKey.cpp: In static member function ‘static bool tmKey::uniToWxk(wxChar, int&, int&)’:
tmKey.cpp:407: error: ‘gdk_unicode_to_keyval’ was not declared in this scope
tmKey.cpp:410: error: ‘gdk_keyval_is_upper’ was not declared in this scope
tmKey.cpp:413: error: ‘gdk_keyval_to_lower’ was not declared in this scope
tmKey.cpp:415: error: ‘wxLogDebug’ was not declared in this scope
tmKey.cpp:419: error: ‘gdk_keyval_to_unicode’ was not declared in this scope
tmKey.cpp:421: error: ‘wxLogDebug’ was not declared in this scope
make: *** [.objs.debug/tmKey.o] Error 1

So in the src/tmKey.cpp file, I added :
#ifdef WXGTK
#include <gdk/gdk.h>
#endif

#include <wx/msgdlg.h>
#include <wx/log.h>

And here come a fresh e.debug executable ^_^

Hi, I followed the linux install instructions and they worked perfectly until the last make command. I get:
:~/projects/e/src$ make DEBUG=1
[LD] e.debug
/usr/bin/ld: skipping incompatible ../ecore/libecored.so when searching for -lecored
/usr/bin/ld: cannot find -lecored

I tried to find libecored and install it, but I was unsuccessful. I looks like it might be in some Enlightenment repositories, but I had no success there either...

In order to unittest Document behavior, you need to create Catalyst objects first. However, when pointing Catalyst to a non-existing e.db, it will attempt to create a modal registration dialog.

In the unittest code, no actual eApp is created, and we don't want any UI to show anyway...so it is impossible to test any thing requiring Catalyst unless you point it at an existing registered e.db.

Using an existing e.db is possible, but not optimal, since it will put the unittest in an unknown starting state (ie, the state that you last left E in when used as an application.)

It may be worthwhile including wxCocoaDialog as an external, using a git submodule.

The Linux build is moving to embed WebKit as the browser used for preview and external script output. Potentially we could move to WebKit on Windows too instead of embedding IE, but I don’t really have a preference.

What is the minimum version of Windows that e should support? The project was set to WINVER=0x4000, which is a 9x/NT4-era value.

To support multi-monitor positioning I'm bumping this up to 0x5000, which means "minimum of WIndows 2000"...is this OK?

Currently, on Ubuntu, there is no e-texteditor icon on the main e window, or on the 'quick launch' button associated with e on the bottom panel.

Suggestion: add this icon, to improve usability

Platform: Ubuntu Jaunty
Version 1.0.34b

Tried the Linux build .35 (http://e-texteditor.com/forum/viewtopic.php?p=13379#13379) . It reports that the trail period is expired, why? I think the Linux version is free?

When running "./build_externals_linux.sh debug" i get this:
In file included from /home/pernissila/appz/e/external/webkit/WebKitTools/wx/../../JavaScriptCore/runtime/JSFunction.h:30,
from /home/pernissila/appz/e/external/webkit/WebKitTools/wx/../../JavaScriptCore/runtime/Lookup.h:26,
from ./bindings/js/JSDOMBinding.h:26,
from ./bindings/js/JSDOMWindowBase.h:24,
from ./DerivedSources/WebCore/JSDOMWindow.h:24,
from ./bindings/js/JSDOMWindowShell.h:32,
from ./bindings/js/ScriptController.h:25,
from ./page/Frame.h:41,
from platform/network/curl/ResourceHandleManager.h:31,
from platform/network/curl/ResourceHandleCurl.cpp:34:
/home/pernissila/appz/e/external/webkit/WebKitTools/wx/../../JavaScriptCore/parser/Nodes.h:2021: internt kompilatorfel: i make_id_declarator, vid cp/parser.c:926
Var vänlig och skicka in en komplett felrapport,
om möjligt med preprocessad källfil.
Se file:///usr/share/doc/gcc-4.3/README.Bugs för instruktioner.
make: *** [obj-gnu/wx_webcore_static_ResourceHandleCurl.o] Fel 1
make: Lämnar katalogen "/home/pernissila/appz/e/external/webkit/WebCore"
Cannot compile WebKit
~/appz/e/external

gcc (Ubuntu 4.3.3-5ubuntu4) 4.3.3

On line 1731, the variable page should be editorCtrl. You get a compile error there if it is not.

It probably has been fixed somewhere, but it is not in the current repository as of July 3rd.

e is using a very old PCRE library (7.6). While trying to build e linking dynamically to PCRE 7.9 I found that ucp.h no longer exists, digging through PCRE changelogs I found:

http://www.pcre.org/changelog.txt (Version 7.8 05-Sep-08)

"Replaced UCP searching code with optimized version as implemented for Ad Muncher (http://www.admuncher.com/) by Peter Kankowski. This uses a two-stage table and inline lookup instead of a function, giving speed ups of 2 to 5 times on some simple patterns that I tested. Permission was given to distribute the MultiStage2.py script that generates the tables (it's not in the tarball, but is in the Subversion repository)"

2 to 5 speed up sounds nice, I have no idea how difficult the ABI changes are to implement.

I'm a C# / Python programmer by day, and each of those platforms has some standard (unit) test harnesses (NUnit, xUnit, MSTest / unittest, doctest).

Anyone have any recommendations for C++ (unit) test frameworks? I'd like to start putting in place some tests, at least for the non-UI parts of the code. Of course I can just "write code" that does work and checks results, but something more organized than "a chunk of code" might be better in the long run.

I am getting the following error, when trying to compile e on ubuntu 8.10 with gcc 4.3.2

../external/out.release/lib/libwxwebkit.a(wxwebkit_static_WebView.o): In function wxStringBase::wxStringBase(wxStringBase const&)': WebView.cpp:(.text._ZN12wxStringBaseC2ERKS_[wxStringBase::wxStringBase(wxStringBase const&)]+0x48): undefined reference towxOnAssert(wchar_t const_, int, char const_, wchar_t const_, wchar_t const_)'
../external/out.release/lib/libwxwebkit.a(wxwebkit_static_WebView.o): In function wxAutoBufferedPaintDC::TestWinStyle(wxWindow*)': WebView.cpp:(.text._ZN21wxAutoBufferedPaintDC12TestWinStyleEP8wxWindow[wxAutoBufferedPaintDC::TestWinStyle(wxWindow*)]+0x4f): undefined reference towxOnAssert(wchar_t const_, int, char const_, wchar_t const_, wchar_t const_)'
../external/out.release/lib/libwebcore-wx.a(wx_webcore_static_StringWx.o): In function String': /home/claudemiro/projetos/e/external/webkit/WebCore/platform/text/wx/StringWx.cpp:80: undefined reference towxOnAssert(wchar_t const_, int, char const_, wchar_t const_, wchar_t const_)'
/home/claudemiro/projetos/e/external/webkit/WebCore/platform/text/wx/StringWx.cpp:80: undefined reference to `wxOnAssert(wchar_t const_, int, char const_, wchar_t const_, wchar_t const_)'
collect2: ld returned 1 exit status
make: *** [e] Error 1

any suggestions?

I'm not sure if this is a widespread problem, but at least on my two windows installations, changes to themes aren't saved.

In the project panel on windows, you can right click on a file or directory to rename, delete, etc.

In Ubuntu, the right click options do not exist.

Suggestions: add a these options, to improve usability

Version: 1.0.34b
Platform: Ubuntu Jaunty

I just run 'make deb' after I got an executable, and it failed.
Here's the message:

$ make deb
[TEST] ../external/stuff contents
make: *** [.test-stuff] Error 1

Doing a build gets you the raw executable file, but the repository currently doesn’t include any of the supporting files or default bundles that a packaged e download does.

If this is by intent (that is, you must get those files from a released e package), then a note to that effect should be added somewhere.

Using WinXP SP3, I've been able to reproduce this using the main branch as well as adamv and joeri's forks. I've yet to go back and see if this occurs on a fresh 1.0.32 install.

Procedure:
I begin by editing a php file (I assume it works with other languages).
Then, I type an if statement or a for statement and hit return.
I'm guessing this is where the problems begin as I would assume the new line should be indented and it is not.
Now, if I type an open bracket, the line indents, the open and close brackets are created, and the cursor is set between them. Everything is copacetic so far...
...right up until I hit enter. That's when everything blows up.

When not debugging, I get an error on FixedLine.cpp(241) which I think is (238) on the main branch. In either case, it's this line:
wxASSERT(!text.empty() && m_extsBuf.size() == text.size());

When debugging, I get a bit more information.
e.exe!FixedLine::SetLine(unsigned int startpos=69, unsigned int endpos=73, bool cache=true) Line 251 + 0x12 bytes C++
e.exe!Lines::SetLine(unsigned int lineId=9) Line 53 + 0x5b bytes C++
e.exe!Lines::SetCaretPos(bool update=false) Line 246 C++
e.exe!Lines::SetPos(unsigned int newpos=71, bool update_lastpos=true) Line 173 C++
e.exe!EditorCtrl::InsertNewline() Line 2142 C++
e.exe!EditorCtrl::InsertChar(const wchar_t & text=L'') Line 2175 C++
e.exe!EditorCtrl::OnChar(wxKeyEvent & event={...}) Line 6591 C++

Apologies if that gets horribly formatted. I'm new to github.

If I move the cursor outside of the brackets, then move it back and hit enter there is no problem. Additionally there are other combination that have the same effect.

I built E on Ubuntu Jaunty and it worked except that whenever I press P, it shows ¨.e/tmcmd:4: undefined method `+' for nil:NilClass (NoMethodError)¨. Anybody knows how to resolve this?

The core library depends on AboutBox having a specific constructor. I'm trying to refactor the AboutBox to remove the dependency on eApp, by having EditorFrame pass in the "version name" instead of the AboutBox trying to get it globally.

Unfortunately, the core library depends on this constructor:
Error 1 error LNK2001: unresolved external symbol "public: __thiscall eAbout::eAbout(class wxWindow *,class Catalyst const &)" (??0eAbout@@QAE@PAVwxWindow@@ABVCatalyst@@@z) ecored.lib e

There are some build warnings:
eApp.cpp
- c:\Projects\e\external\libtomcrypt\src\headers\tomcrypt_cfg.h(11) : warning C4565: ‘malloc’ : redefinition; the symbol was previously declared with __declspec(restrict)
- c:\Projects\e\external\libtomcrypt\src\headers\tomcrypt_cfg.h(12) : warning C4565: ‘realloc’ : redefinition; the symbol was previously declared with __declspec(restrict)
- c:\Projects\e\external\libtomcrypt\src\headers\tomcrypt_cfg.h(13) : warning C4565: ‘calloc’ : redefinition; the symbol was previously declared with __declspec(restrict)
- c:\Projects\e\external\libtomcrypt\src\headers\tomcrypt_cfg.h(14) : warning C4565: ‘free’ : redefinition; the symbol was previously declared with __declspec(noalias)
- c:\Projects\e\external\wxwidgets\include\wx/datetime.h(1108) : warning C4996: ‘localtime’: This function or variable may be unsafe. Consider using localtime_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
- C:\Program Files\Microsoft Visual Studio 9.0\VC\include\time.inl(114) : see declaration of ‘localtime’

(Duplicates removed.)

e.exe still builds and runs, seemingly OK, but it would be nice to have the build be as warning-free as possible.

(Note, I’m bad at Markdown formatting.)

In EditorFrame::GetEditorCtrlFromFile(const wxString&, unsigned int&) :

It currently is:
if (filepath == page->GetPath()) {

It should be:
if (filepath == editorCtrl->GetPath()) {

(I didn't think it was worth a fork...)

I run Slackaware 12.2. I installed the dependencies given in the linux-notes.txt file.
I ran get_external_linux script and the build_external_linux script with the debug option.
This works fine, and the final message is as follows

.

I opened e.cfg in e, then closed.
Whenever I open e again, it asks me if I want to reload e.cfg. Since this is the initial open, the answer is "of course I do." I've noticed this with other files in the past, but having e.cfg in the open files list on startup really makes it prominent.

It would be great to have official GitHub repos for support/bundles/cocoadialog, so that we can contribute fixes to those as well. I've already posted a fix or two to cocoadialog, for instance.

When running the trial version on Windows, if the editor is never closed, the trial never ends. I have had E running constantly for 5 days and the title bar says I still have 29 days left.

Hi.

I'm on Ubuntu Jaunty 64bits. I pulled e today (1st May). When compiling e I ran into this error :

 [CC] tm_syntaxhandler.cpp
 In file included from ../external/out.debug/include/mk4.h:1053,
                       from plistHandler.h:21,
                       from tm_syntaxhandler.h:24,
                       from tm_syntaxhandler.cpp:14:
 ../external/out.debug/include/mk4.inl: In function ‘bool operator<(c4_Cursor, c4_Cursor)’:
 ../external/out.debug/include/mk4.inl:287: warning: suggest parentheses around && within ||
 tm_syntaxhandler.cpp: In static member function ‘static bool tmKey::wxkToUni(int, bool, wxChar&)’:
 tm_syntaxhandler.cpp:2200: error: ‘gdk_unicode_to_keyval’ was not declared in this scope
 tm_syntaxhandler.cpp:2203: error: ‘gdk_keyval_to_upper’ was not declared in this scope
 tm_syntaxhandler.cpp:2203: error: ‘gdk_keyval_to_lower’ was not declared in this scope
 tm_syntaxhandler.cpp:2204: error: ‘gdk_keyval_to_unicode’ was not declared in this scope
 tm_syntaxhandler.cpp: In static member function ‘static bool tmKey::uniToWxk(wxChar, int&, int&)’:
 tm_syntaxhandler.cpp:2233: error: ‘gdk_unicode_to_keyval’ was not declared in this scope
 tm_syntaxhandler.cpp:2236: error: ‘gdk_keyval_is_upper’ was not declared in this scope
 tm_syntaxhandler.cpp:2239: error: ‘gdk_keyval_to_lower’ was not declared in this scope
 tm_syntaxhandler.cpp:2245: error: ‘gdk_keyval_to_unicode’ was not declared in this scope
 make: *** [.objs.debug/tm_syntaxhandler.o] Error 1

libgtk+2.0-dev is installed. The gdk_keyval* functions are in /usr/include/gtk-2.0/gdk/gdkkeys.h
Please tell me if you need any information about my configuration or system.

Best regards.

Different close button graphics are used for the panels (bundle pane, project pane, search pane, etc.) and the editor tabs.

It's not bad per se that these graphics are different, but that they already look pretty similar and yet are different.

I run Slackaware 12.2. I installed the dependencies given in the linux-notes.txt file.
I ran get_external_linux script and the build_external_linux script with the debug option.
This works fine, and the final message is as follows

--- BUILD COMPLETE ---

Next steps:

-- Run '/home/riodeuno/e/external/webkit/WebKitTools/wx/../../WebKitTools/Scripts/run-javascriptcore-tests --wx' to ensure JSCore tests pass.

-- Run /home/riodeuno/e/external/webkit/WebKitBuild/Release/wxBrowser to test your wxWebKit build.

Both of the above test fail. First test fails with this error

Running: build-jsc --wx --release --32-bit --qt --wx
Died at /home/riodeuno/e/external/webkit/WebKitTools/wx/../../WebKitTools/Scripts/run-javascriptcore-tests line 135.

The second test executable does not exist.

A Makefile is generated in the ../src folder, when trying to run "make"
The following error shows and make stops

RemoteThread.h:123: error: class 'RemoteThread::RemoteAction' does not have any field named 'm_url'
RemoteThread.h:123: error: invalid use of incomplete type 'const struct wxString'
SymbolRef.h:4: error: forward declaration of 'const struct wxString'
RemoteThread.h:123: error: class 'RemoteThread::RemoteAction' does not have any field named 'm_login'
RemoteThread.h:123: error: invalid use of incomplete type 'struct wxString'
SymbolRef.h:4: error: forward declaration of 'struct wxString'
RemoteThread.h:123: error: class 'RemoteThread::RemoteAction' does not have any field named 'm_target'
RemoteThread.h:123: error: invalid use of incomplete type 'const struct wxString'
SymbolRef.h:4: error: forward declaration of 'const struct wxString'
RemoteThread.h: At global scope:
RemoteThread.h:201: error: expected constructor, destructor, or type conversion before 'DECLARE_EVENT_TYPE'
make: *** [.objs.release/BundleItemEditorCtrl.o] Error 1

Is there something i am missing?

I can't find any client code using the mod or scroll callbacks exposed by EditorCtrl. Safe to delete these?

linking errors on e.debug
http://saidin.pastebin.com/f6505974a

I have attached a patch for this. it is outputting FF characters rather than CR in crlf / UTF16 encoding.

The header has that guard, and so should this file

I found a few additional packages that are needed in order to get this to compile under Jaunty (9.04) after following the linux-notes.

libcurl4-dev (yes, I know it says you need libcurl-dev ... but that package doesn't seem to exist in jaunty)
libxml2-dev
gtk-doc-tools (this might be a meta package & have too much other stuff... but I couldn't find a specific package with "gtkdocize" in a shorter period of time)
libsoup2.4-dev
libgstreamer0.10-dev
libgstreamer-plugins-base0.10-dev

most of these might just be dependents used by the wx stuff... so they may not directly relevant to E's source... but I couldn't get it to compile without them.

Building off the master branch fails. Commit information:

$ git rev-parse HEAD
76670fd

make DEBUG=1

.. elided ..
plistHandler.cpp: In member function ‘wxString PListHandler::GetSyntaxAssoc(const wxString&) const’:
plistHandler.cpp:232: error: operands to ?: have different types ‘const wxChar*’ and ‘wxString’

"BundleEditor" has been deprecated in favor of "BundlePane"; BundleEditor and any support code used by it alone should be removed from the .vcproj and file system.

(Is there still code in BundleEditor that is pending to migrate into BundlePane?)

Let's have two files: one typed and saved in E, second typed and saved in any other editor, and then loaded to E.

In first file all brackets highlighted correctly and autocomplete (on ESC key) work, but on second file brackets highlighting not work at all and autocomplete also didn't work.

Screenshot: http://images.detstwo.com/view/soRj44.png

(I can try to fix this bug, but I don't know what files/classes related to thit problem)

Since doc_byte_iter is a friend of Document, but is closed source, it is preventing some refactoring of Document's internals.

It would be helpful, though not essential, for doc_byte_iter to use only public methods of Document if possible, but I understand if performance or other considerations prevent this from being possible.

I'm compiling using Ubuntu 9.04 and following the linux-notes, did I forget to do something?
BTW, I noticed the file size of libecored.so (679K) is much smaller than ecored.lib (5.9M), is that correct?

Part of the output below:

[LD] e.debug
.objs.debug/BundleManager.o: In function BundleManager::AddItems(wxString const&, std::vector<cxFileInfo, std::allocator<cxFileInfo> > const&)': /home/houdw/dev/e/src/BundleManager.cpp:177: undefined reference toCatalyst::GetDateAge(wxDateTime const&)'
.objs.debug/Cell.o: In function Document::GetReadLock() const': Cell.cpp:(.text._ZNK8Document11GetReadLockEv[Document::GetReadLock() const]+0xf): undefined reference toCatalyst::GetReadLock() const'
.objs.debug/cx_pcre_exec.o: In function match': /home/houdw/dev/e/src/cx_pcre_exec.cpp:518: undefined reference todoc_byte_iter::doc_byte_iter()'
/home/houdw/dev/e/src/cx_pcre_exec.cpp:520: undefined reference to doc_byte_iter::doc_byte_iter()' /home/houdw/dev/e/src/cx_pcre_exec.cpp:604: undefined reference todoc_byte_iter::operator=(doc_byte_iter const&)'
/home/houdw/dev/e/src/cx_pcre_exec.cpp:645: undefined reference to doc_byte_iter::operator=(doc_byte_iter const&)' /home/houdw/dev/e/src/cx_pcre_exec.cpp:852: undefined reference todoc_byte_iter::operator=(doc_byte_iter const&)'
/home/houdw/dev/e/src/cx_pcre_exec.cpp:862: undefined reference to doc_byte_iter::operator=(doc_byte_iter const&)' /home/houdw/dev/e/src/cx_pcre_exec.cpp:864: undefined reference todoc_byte_iter::operator=(doc_byte_iter const&)'
.objs.debug/cx_pcre_exec.o:/home/houdw/dev/e/src/cx_pcre_exec.cpp:968: more undefined references to doc_byte_iter::operator=(doc_byte_iter const&)' follow .objs.debug/cx_pcre_exec.o: In functionmatch':
/home/houdw/dev/e/src/cx_pcre_exec.cpp:1306: undefined reference to doc_byte_iter::operator[](int) const' /home/houdw/dev/e/src/cx_pcre_exec.cpp:1306: undefined reference todoc_byte_iter::operator const'
/home/houdw/dev/e/src/cx_pcre_exec.cpp:1331: undefined reference to doc_byte_iter::operator=(doc_byte_iter const&)' /home/houdw/dev/e/src/cx_pcre_exec.cpp:1342: undefined reference todoc_byte_iter::operator const'
/home/houdw/dev/e/src/cx_pcre_exec.cpp:1342: undefined reference to doc_byte_iter::operator[](int) const' /home/houdw/dev/e/src/cx_pcre_exec.cpp:1353: undefined reference todoc_byte_iter::operator const'
/home/houdw/dev/e/src/cx_pcre_exec.cpp:1353: undefined reference to doc_byte_iter::operator[](int) const' /home/houdw/dev/e/src/cx_pcre_exec.cpp:1372: undefined reference todoc_byte_iter::operator const'

...

.objs.debug/RevTooltip.o: In function RevTooltip::SetDocument(doc_id const&, wxPoint)': /home/houdw/dev/e/src/RevTooltip.cpp:95: undefined reference toCatalyst::IsOk(doc_id const&) const'
/home/houdw/dev/e/src/RevTooltip.cpp:100: undefined reference to Catalyst::GetDocAuthor(doc_id const&) const' /home/houdw/dev/e/src/RevTooltip.cpp:101: undefined reference toCatalyst::GetUserPic(unsigned int) const'
/home/houdw/dev/e/src/RevTooltip.cpp:102: undefined reference to Catalyst::GetUserName(unsigned int) const' /home/houdw/dev/e/src/RevTooltip.cpp:105: undefined reference toCatalyst::GetUserColor(unsigned int) const'
/home/houdw/dev/e/src/RevTooltip.cpp:110: undefined reference to Catalyst::GetLabel(doc_id const&) const' /home/houdw/dev/e/src/RevTooltip.cpp:115: undefined reference toCatalyst::GetDocDate(doc_id const&) const'
/home/houdw/dev/e/src/RevTooltip.cpp:116: undefined reference to Catalyst::GetDocAge(doc_id const&) const' /home/houdw/dev/e/src/RevTooltip.cpp:121: undefined reference toCatalyst::GetDescription(doc_id const&) const'
/home/houdw/dev/e/src/RevTooltip.cpp:124: undefined reference to Catalyst::IsMirrored(doc_id const&) const' /home/houdw/dev/e/src/RevTooltip.cpp:128: undefined reference toCatalyst::GetMirrorPathsForDraft(doc_id const&, wxArrayString&) const'
/home/houdw/dev/e/src/RevTooltip.cpp:129: undefined reference to Catalyst::GetMirrorPaths(doc_id const&, wxArrayString&) const' .objs.debug/SettingsDlg.o: In functionSettingsDlg::OnButtonOk(wxCommandEvent&)':
/home/houdw/dev/e/src/SettingsDlg.cpp:358: undefined reference to Catalyst::GetUserName(unsigned int) const' /home/houdw/dev/e/src/SettingsDlg.cpp:365: undefined reference toCatalyst::SetProfile(wxString const&, wxImage const&)'
.objs.debug/SettingsDlg.o: In function SettingsDlg': /home/houdw/dev/e/src/SettingsDlg.cpp:168: undefined reference toCatalyst::GetUserName(unsigned int) const'
/home/houdw/dev/e/src/SettingsDlg.cpp:176: undefined reference to Catalyst::GetUserPic(unsigned int) const' /home/houdw/dev/e/src/SettingsDlg.cpp:168: undefined reference toCatalyst::GetUserName(unsigned int) const'
/home/houdw/dev/e/src/SettingsDlg.cpp:176: undefined reference to Catalyst::GetUserPic(unsigned int) const' .objs.debug/styler_users.o: In functionStyler_Users::Style(StyleRun&)':
/home/houdw/dev/e/src/styler_users.cpp:21: undefined reference to Catalyst::GetDocAuthor(doc_id const&) const' /home/houdw/dev/e/src/styler_users.cpp:25: undefined reference toCatalyst::GetUserColor(unsigned int) const'
.objs.debug/styler_users.o: In function Document::GetNodeInfo(unsigned int) const': /home/houdw/dev/e/src/Document.h:104: undefined reference toDataText::GetNodeInfo(unsigned int) const'
.objs.debug/UndoHistory.o: In function UndoHistory::OnVersionTreeDrawitem(VersionTreeEvent&)': /home/houdw/dev/e/src/UndoHistory.cpp:546: undefined reference toCatalyst::IsMirroredSpecific(doc_id const&) const'
.objs.debug/UndoHistory.o: In function UndoHistory::OnVersionTreeTooltip(VersionTreeEvent&)': /home/houdw/dev/e/src/UndoHistory.cpp:524: undefined reference toCatalyst::IsMirroredSpecific(doc_id const&) const'
/home/houdw/dev/e/src/UndoHistory.cpp:526: undefined reference to Catalyst::GetMirrorPaths(doc_id const&, wxArrayString&) const' .objs.debug/UndoHistory.o: In functionUndoHistory::OnDocUpdated(UndoHistory_, void_, int)':
/home/houdw/dev/e/src/UndoHistory.cpp:618: undefined reference to `Catalyst::InSameHistory(doc_id const&, doc_id const&) const'
collect2: ld returned 1 exit status
make: *** [e.debug] Error 1