Server Help

Cyan~Fire

OK, I have to revamp a bit of WM_COMMAND-processing code I wrote for a new feature in a program I'm writing.

Now, it just so happened before that I didn't have to process a BN_CLICKED and a menu item click before, but now that I do, is it true that they both have a notification code of 0?!

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

All commands should be handled by code AND id, otherwise you can confuse lots of messages. You can differentiate between menus, accelerators, and control notifications.
_________________
4,691 irradiated haggis!

Cyan~Fire · Posted: Sat Apr 09, 2005 1:56 pm *Post maybe stupid* Post subject:

Yeah, don't worry, I haven't been doing it just by code, I've been doing it by MAKELONG() with the full wParam. But sadly I can't do that now, since I'm processing the EN_SETFOCUS and EN_KILLFOCUS messages, and I somehow don't want to make a case for every single edit control I have.

What I find messy is simply this:

Code: Show/Hide

switch (code)
{
case BN_CLICKED: //and menuitem click!
case CBN_SELCHANGE: //and accelerator!
switch (id)

I don't see why MS couldn't just allocate a range of numbers for each control. I mean, I really doubt that there are more than 65535 notifications, and the newer ones use WM_NOTIFY.

Anyway, thanks.

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

Yes I have often thought the same thing. In my command handler, I use if/else-if/else-if/else type statements. Doesn't have to be super efficient because commands are mostly at UI speed (ie user keys and clicks).

If you were ambitious, you could also combine code/id into a single value and switch on that.

Cyan~Fire · Posted: Sat Apr 09, 2005 3:53 pm *Post maybe stupid* Post subject:

Mine GO BOOM · Posted: Sun Apr 10, 2005 1:41 am *Post maybe stupid* Post subject:

Solo Ace · Posted: Sun Apr 10, 2005 3:48 pm *Post maybe stupid* Post subject:

CypherJF · Posted: Sun Apr 10, 2005 5:14 pm *Post maybe stupid* Post subject:

I didn't even know where to begin, most of the pages I lookup uses MFC or some variant of it.
_________________
Performance is often the art of cheating carefully. - James Gosling

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

MFC is like a virus. It took over Win32 API without improving it. It now dominates all searches for any real info.

Smong · Posted: Sun Apr 10, 2005 8:53 pm *Post maybe stupid* Post subject:

I had a go at some GUI stuff last weekend. I had to port my code to c++ since that's what wxWidgets uses. It made the program 5 times bigger.
When I got round to installing the wx devpak in devcpp the barebones program int the project template is 1.8mb (statically linked I believe). The gtk libs were over 6mb themselves so wx is an improvement in a way.

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

My win32 apps (statically linked) are usually in the 20-60K area, unless they have a ton of resources in them.

Cyan~Fire · Posted: Sun Apr 10, 2005 11:00 pm *Post maybe stupid* Post subject:

Eek. My program is 100K! It's dynamically linked, which by my understanding, should make it smaller? What's wrong?

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

Straight win32? No other non-OS libs? No wrappers like MFC, etc? Large resources? Release mode compile?

CypherJF · Posted: Mon Apr 11, 2005 1:40 pm *Post maybe stupid* Post subject:

Cyan~Fire · Posted: Mon Apr 11, 2005 5:46 pm *Post maybe stupid* Post subject:

Well, I am using zlib.

Code: Show/Hide

## Makefile for AOK Trigger Studio by David Tombs

#Check for CFG macro

!IF "$(CFG)" == ""
!MESSAGE No configuration specified. Defaulting to Win32 Debug.
!MESSAGE
CFG=Win32 Debug
!ENDIF

#Release build
!IF "$(CFG)" == "Win32 Release"

OUTDIR=.
INTDIR=.\Release

CPPFLAGS_OPT = -ML -O2
LINK32_FLAGS = /release /incremental:no

#Debug build
!ELSEIF "$(CFG)" == "Win32 Debug"

OUTDIR=.\Debug
INTDIR=.\Debug

CPPFLAGS_DEBUG = -MLd -Gm -D "_DEBUG" -FAs -Fa"asm/" -Zi -Fd"$(INTDIR)\\"
LINK32_FLAGS = /debug /pdbtype:sept
RFLAGS_DEBUG = /D "_DEBUG"

!ELSE

!ERROR Invalid configuration "$(CFG)" specified.

!ENDIF

LINK_LIBS=zlib.lib user32.lib comdlg32.lib comctl32.lib advapi32.lib
CPPFLAGS = -c $(CPPFLAGS_DEBUG) $(CPPFLAGS_OPT) -YX -Fo"$(INTDIR)\\" -D WINVER=0x0400 -EHsc -W3
RFLAGS = /fo "$(INTDIR)\AOKTS.res" $(RFLAGS_DEBUG)

LINK_OBJS = \
$(INTDIR)/aokts.obj \
$(INTDIR)/datatypes.obj \
$(INTDIR)/editors.obj \
$(INTDIR)/scen.obj \
$(INTDIR)/scen_const.obj \
$(INTDIR)/trigedit.obj \
$(INTDIR)/ecedit.obj \
$(INTDIR)/utilui.obj \
$(INTDIR)/zlibfile.obj \
$(INTDIR)/unitedit.obj \
$(INTDIR)/trigger.obj \
$(INTDIR)/utilio.obj \
$(INTDIR)/AOKTS.res

#Inference Rules

.cpp{$(INTDIR)}.obj:
cl $(CPPFLAGS) $<

.rc{$(INTDIR)}.res:
rc $(RFLAGS) $<

#Targets

build: $(OUTDIR)\aokts.exe
@echo Build complete.

source: $(OUTDIR)\source.zip
@echo Source zipped.

#Build targets

$(OUTDIR)\aokts.exe: $(LINK_OBJS)
link.exe $(LINK32_FLAGS) /out:"$@" /pdb:"$(OUTDIR)\aokts.pdb" $(LINK_LIBS) $(LINK_OBJS)

$(LINK_OBJS):

#Source targets: yes, this is a stupid version of a batch file.

TEMPFILES=aokts.cpp datatypes.cpp editors.cpp scen.cpp scen_const.cpp trigedit.cpp utilui.cpp zlibfile.cpp ecedit.cpp unitedit.cpp

$(OUTDIR)\source.zip: *.cpp *.h aokts.rc src_notes.txt MAKEFILE res\aokts_big.bmp res\aokts.ico
@"C:\Program Files\7-zip\7z" u $(OUTDIR)\source.zip $**

!INCLUDE makefile.dep

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

My static zlib.lib is 66K in release mode. It's a build I made with just the stuff I've used. For example, I don't use the file stuff. I have no idea how much bigger this lib makes the EXE though.

Cyan~Fire · Posted: Mon Apr 11, 2005 6:30 pm *Post maybe stupid* Post subject:

The dynamic one with all the functions I'm using is 10.5K.

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

Dynamic means the EXE will use an external DLL.

Cyan~Fire · Posted: Mon Apr 11, 2005 7:52 pm *Post maybe stupid* Post subject:

But it still needs a lib to link to, right? I looked in zlib.lib with a hex editor, and it seems it's just a bunch of function names and offsets.

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

A dynamic lib simply lists the exports that are in the DLL. This allows the linker to know that they exist, and to place the appropriate references in the EXE header so the loader knows to load zlib.dll and what functions to find/resolve.

Cyan~Fire · Posted: Mon Apr 11, 2005 8:45 pm *Post maybe stupid* Post subject:

OK, that's what I thought. So... any clue what's going on? The source itself is 160K, if that's around the range you're looking for.

Mr Ekted · Movie Geek Gender: Joined: Feb 09 2004 Posts: 1379 Offline

You use anything gay like STL?

Cyan~Fire · Posted: Tue Apr 12, 2005 3:55 pm *Post maybe stupid* Post subject:

I don't use STL, but I use one template class that I wrote. I guess that does take up a bit of room, as it's used for 4 different classes, so 4 different copies.

But it's nowhere near STL which includes around 10 templates for one you use in your code.

Cyan~Fire · Posted: Mon Jun 20, 2005 9:05 pm *Post maybe stupid* Post subject:

Off topic, but still Stupid Windows API.

Any way to catch a return sent to an edit control other than by subclassing? Of course, only common controls send NM_RETURN (not to mention WM_NOTIFY in general, which only makes command processing 50 times easier).