plib-devel

Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection bug, make not needed sliders disappear

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

Melchior,

	I'm putting in your first puAuxLargeInput patch right now.

	Do me a favor and send the other ones separately.

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Melchior FRANZ [mailto:mf...@us...] 
Sent: Tuesday, January 20, 2009 1:31 PM
To: pli...@li...
Subject: Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection
bug,make not needed sliders disappear

* Fay, John F Dr CTR USAF AFMC 46 SK -- Tuesday 20 January 2009:
> Is it all one change or can it be broken up into pieces?

It's two things: a minor fix (which is only two lines, so factoring
it out doesn't make a lot of sense), and the rest is one feature
(with a bit of cleanup and abstraction).

The feature is "autohiding sliders". They are hiding, if they aren't
necessary, because the lines are short enough for the width, and
few enough for the box height. This doesn't only look nicer, it also
frees precious space and is kind of a standard in modern GUIs.

With shorter response times I could make smaller patches, but if
getting one patch in takes five days, then this is impractical.
I can do a lot in five days, and there is too much overlapping
to write subsequent patches. What if you make changes to the first?
Then all others are likely no longer applicable. I know, of course,
that you are the only left-over committer (just as I'm the only
left-over pui/puAux developer :-), and I'm happy to get something
reviewed and committed at all.

m.


PS: Here's my testing setup in FlightGear:
    http://members.aon.at/mfranz/fgfs-plib.jpg  [98.3 kB]
    The "windows" are all resizable, so autohiding and line
    wrapping are easy to observe.

------------------------------------------------------------------------
------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
plib-devel mailing list
pli...@li...
https://lists.sourceforge.net/lists/listinfo/plib-devel

[Plib-devel] [RFC] changing slider behavior

[Melchior F. <mf...@us...>]

Sliders (puSlider, puaScrollbox) work in a very unconventional
way now. The reference point for moving the slider handle is
always in the middle of the handle (A)...

  +---+------------+=============+--------------+---+
  | < |            | B    A      |              | > |
  +---+------------+=============+--------------+---+
           bgnd        handle         bgnd

If you click anywhere on the slider, the handle jumps there,
in a way that the handle is centered under the mouse. If you
grab the handle at 'B', then the slider will first jump to
the left, and can only then get adjusted. That's weird and
unlike any slider behavior in one of today's GUIs (that I
know). If the handle was at one end, then you have to move
the mouse first until you reach the center point, before the
slider even starts moving! Weird and unexpected!

In any other GUI it's like this (as you probably know):
You can grab the slider at any point and start moving. No
jumps at all. If you click on the background left and
right of the slider, then it makes a big jump ("page_step"),
and on the arrow buttons (if available) it makes a small
step ("line_step"). I'd like to change puSlider/puaScrollbar
to this behavior (and puaList, puaLargeInput etc. would
inherit it). Actually, the patch for that is already finished
and waits for submission.

Does anyone have an objection to this change? If not, should
the old style be available as an alternative? As default?

m.

Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection bug, make not needed sliders disappear

[Melchior F. <mf...@us...>]

* Melchior FRANZ -- Tuesday 20 January 2009:
> With shorter response times I could make smaller patches, but if
> getting one patch in takes five days, then this is impractical.

BTW: that wasn't the explanation for this patch being big, but
for the next being big again. It'll be a bit easier to cut in
pieces, though, so there's still hope.  :-)

m.

Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection bug, make not needed sliders disappear

[Melchior F. <mf...@us...>]

* Fay, John F Dr CTR USAF AFMC 46 SK -- Tuesday 20 January 2009:
> Is it all one change or can it be broken up into pieces?

It's two things: a minor fix (which is only two lines, so factoring
it out doesn't make a lot of sense), and the rest is one feature
(with a bit of cleanup and abstraction).

The feature is "autohiding sliders". They are hiding, if they aren't
necessary, because the lines are short enough for the width, and
few enough for the box height. This doesn't only look nicer, it also
frees precious space and is kind of a standard in modern GUIs.

With shorter response times I could make smaller patches, but if
getting one patch in takes five days, then this is impractical.
I can do a lot in five days, and there is too much overlapping
to write subsequent patches. What if you make changes to the first?
Then all others are likely no longer applicable. I know, of course,
that you are the only left-over committer (just as I'm the only
left-over pui/puAux developer :-), and I'm happy to get something
reviewed and committed at all.

m.


PS: Here's my testing setup in FlightGear:
    http://members.aon.at/mfranz/fgfs-plib.jpg  [98.3 kB]
    The "windows" are all resizable, so autohiding and line
    wrapping are easy to observe.

Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection bug, make not needed sliders disappear

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

Melchior,

	Pardon my silence, I've been thoroughly slammed (that means
busy, not drunk or arrested) in the last several days.  I looked at your
first change and it looks pretty extensive.  Is it all one change or can
it be broken up into pieces?

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Melchior FRANZ [mailto:mf...@us...] 
Sent: Monday, January 19, 2009 3:50 PM
To: pli...@li...
Subject: Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection
bug,make not needed sliders disappear

* Melchior FRANZ -- Friday 16 January 2009:
> Next planned steps:
> 
> - fix puSlider and puaScrollbar (don't assume reference point
>   in the middle of the slider)

That's now done, but I can't submit it as long as this patch
isn't reviewed/applied. Or should I submit one big patch with
both changes? I'll be starting with the next patch now ...

m.

------------------------------------------------------------------------
------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
plib-devel mailing list
pli...@li...
https://lists.sourceforge.net/lists/listinfo/plib-devel

Re: [Plib-devel] [PATCH] puAuxLargeInput: fix selection bug, make not needed sliders disappear

[Melchior F. <mf...@us...>]

* Melchior FRANZ -- Friday 16 January 2009:
> Next planned steps:
> 
> - fix puSlider and puaScrollbar (don't assume reference point
>   in the middle of the slider)

That's now done, but I can't submit it as long as this patch
isn't reviewed/applied. Or should I submit one big patch with
both changes? I'll be starting with the next patch now ...

m.

[Plib-devel] [PATCH] puAuxLargeInput: fix selection bug, make not needed sliders disappear

[Melchior F. <mf...@us...>]

Here's another feature & bugfix patch:

- don't draw selection rectangles outside the input box
- make sliders disappear if they aren't required
- fill the square hole in the lower right corner with a "plug"
  if necessary (-> both sliders shown)
- remove methods that were marked as "deprecated" since years
- remove trailing spaces, replace tabs by spaces, other cosmetics



Next planned steps:

- fix puSlider and puaScrollbar (don't assume reference point
  in the middle of the slider)
- autoscrolling selection in puaLargeInput? More cleanup ...

m.

Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw outside the box, fix sliders

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

OK ... I'm reading my e-mails out of order.  I did indeed find the
errant "cerr" and removed it.

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Melchior FRANZ [mailto:mf...@us...] 
Sent: Monday, January 12, 2009 1:57 PM
To: pli...@li...
Subject: Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw
outsidethe box, fix sliders

BTW: I'd like to make some more changes, but prefer to have
that reviewed and committed in steps, so there may be one or
two further patches in the next days. But every one of them
should be an improvement.

m.


PS: I hope you found the cerr that I accidently left in the
    code. Please remove.  :-)

------------------------------------------------------------------------
------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
plib-devel mailing list
pli...@li...
https://lists.sourceforge.net/lists/listinfo/plib-devel

Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw outside the box, fix sliders

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

Melchior,

	I'm putting in the patch and found these two lines in
"puauxLargeInput.cxx" around line 395:

+    if (bottom_slider)
+        cerr << bottom_value << endl;

With your permission I will not put these two lines into the release
code.

	I'm putting the other changes into SVN.  Again, please check
them out to make sure I haven't messed anything up.

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Melchior FRANZ [mailto:mf...@us...] 
Sent: Monday, January 12, 2009 1:28 PM
To: pli...@li...
Subject: Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw
outsidethe box, fix sliders

Argh ... forgot the puAux.h patch. So here a complete version:  ;-)

- don't write outside of the text box
- fix slider fractions
- remove redundant code

m.

Re: [Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

Melchior,

	No defense is necessary, you are not under attack.  In fact, you
have my thanks for using the library and taking the trouble to fix the
bugs.

	I've put the changes in as two separate commits--one for the
blank lines and indentation and the other with the actual changes.
Please make sure I've done it correctly.

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Melchior FRANZ [mailto:mf...@us...] 
Sent: Monday, January 12, 2009 1:23 PM
To: pli...@li...
Subject: Re: [Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

To my defense: I have, of course, tested the previous patch, and it
worked for hours ... then I suddenly got a SIGFPE.
Take the attached version -- it also fixes some bad indentation and
removes trailing spaces.

"Allow slider fractions of 1.0 -- don't force them to 0.9."

m.

Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw outside the box, fix sliders

[Melchior F. <mf...@us...>]

BTW: I'd like to make some more changes, but prefer to have
that reviewed and committed in steps, so there may be one or
two further patches in the next days. But every one of them
should be an improvement.

m.


PS: I hope you found the cerr that I accidently left in the
    code. Please remove.  :-)

Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw outside the box, fix sliders

[Melchior F. <mf...@us...>]

Argh ... forgot the puAux.h patch. So here a complete version:  ;-)

- don't write outside of the text box
- fix slider fractions
- remove redundant code

m.

Re: [Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw outside the box

[Melchior F. <mf...@us...>]

Here's a new patch for this, too:

- don't write outside of the text box
- fix slider fractions
- remove redundant code

m.

Re: [Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

[Melchior F. <mf...@us...>]

To my defense: I have, of course, tested the previous patch,
and it worked for hours ... then I suddenly got a SIGFPE.
Take the attached version -- it also fixes some bad indentation
and removes trailing spaces.

"Allow slider fractions of 1.0 -- don't force them to 0.9."

m.

Re: [Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

[Fay, J. F Dr C. U. A. 46 S. <joh...@eg...>]

OK ... I'll wait.  Good luck.

John F. Fay
Technical Fellow
Jacobs Technology TEAS Group
850-883-1294 

-----Original Message-----
From: Melchior FRANZ [mailto:mf...@us...] 
Sent: Monday, January 12, 2009 12:51 PM
To: pli...@li...
Subject: Re: [Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

Don't commit! This code can't even handle a fraction of 1.0.
Shudder. Need to fix that first. Sigh ...

m.


------------------------------------------------------------------------
------
This SF.net email is sponsored by:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
plib-devel mailing list
pli...@li...
https://lists.sourceforge.net/lists/listinfo/plib-devel

Re: [Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

[Melchior F. <mf...@us...>]

Don't commit! This code can't even handle a fraction of 1.0.
Shudder. Need to fix that first. Sigh ...

m.

[Plib-devel] [PATCH] puSlider.cxx: allow fraction 1.0

[Melchior F. <mf...@us...>]

Currently puSlider changes slider fractions >=1 to 0.9. That's
patronizing and very annoying. A fraction of 1.0 makes perfectly
sense, and if some puSlider using code doesn't like that, it can
still replace 1.0 by 0.9 or M_PI.  (Of course, ideally, a slider
should get removed if the fraction is 1.0, but this is, again,
for the plib user to decide, not plib.)

m.

[Plib-devel] [PATCH] puAuxLargeInput.cxx: don't draw outside the box

[Melchior F. <mf...@us...>]

Currently up to 1.5 lines are drawn outside the box. One line
is due to an off-by-one bug in the draw() method, and the half
one is because the top margin isn't considered. The patch also
removes a comment about code that is no longer there.

m.

Re: [Plib-devel] [Plib-users] PLIB-1.8.5 make errors

[John F. F. <joh...@cy...>]

Whoops, typo on my part.  I had said "UL_GYGWIN" instead of "UL_CYGWIN" 
when I was testing the fix.  The library will indeed build with the change 
that I gave in my earlier missive.

	- John


-----Original Message-----
From:	John F. Fay
Sent:	Thursday, January 01, 2009 10:06 AM
To:	'PLIB Users'
Cc:	'PLIB Developers'
Subject:	Re: [Plib-devel] [Plib-users] PLIB-1.8.5 make errors


DB,

	Unfortunately I am not an expert on either SSG or cygwin.  I downloaded
cygwin a couple of months ago and tried building PLIB in it so that I can
reproduce your error.  After some adventures with "dos2unix" (I'm trying to 
build the SVN version of PLIB, which I downloaded using a Windows
TortoiseSVN version), I'm running "configure."

	The root of your problem is a "#define" on line 243 of the offending file. 
 Windows, in the infinite wisdom of Microsoft, decided that "stat" needed
to be "_stat" and the rest of the world is living with the consequences.
 An immediate fix to your problem is tochange the "#if" on line 242 of
"ssgLoadTexture.cxx" to the following:

		#if defined(UL_WIN32) && !defined(UL_CYGWIN)

Unfortunately that doesn't actually solve the problem.  Through a chain of
inclusions, we get back to "util/ul.h" which defines "UL_CYGWIN" if the
symbol "__CYGWIN__" is defined.  Apparently the latter symbol is not
defined, and somebody with more knowledge than I needs to put something in
the Makefile to define it.

		- John


-----Original Message-----
From:	DB
Sent:	Sunday, December 28, 2008 3:01 AM
To:	pli...@li...
Subject:	[Plib-users] PLIB-1.8.5 make errors

Hi,

I've downloaded the current stable plib-1.8.5 and performed the following
operations in cygwin:
 ./configure --prefix=/fg-1.9.0
and make.

but something went wrong.
I received the following error:
 "
make[2]: Entering directory `/home/IBM User/tar/plib-1.8.5/src/ssg'
if g++ -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\"
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"plib\"
-DVERSION=\
"1.8.5\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1
-DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1
-DHAVE_INT
TYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DWIN32=1 -I. -I.
-I../../src/sg -I../../src/util    -g -O2 -Wall -MT ssgLoadTexture.o -MD
-MP -MF ".dep
s/ssgLoadTexture.Tpo" -c -o ssgLoadTexture.o ssgLoadTexture.cxx; \
        then mv -f ".deps/ssgLoadTexture.Tpo" ".deps/ssgLoadTexture.Po";
else rm -f ".deps/ssgLoadTexture.Tpo"; exit 1; fi
ssgLoadTexture.cxx: In function `int fileMTimeCmp(const char*, const
char*)':
ssgLoadTexture.cxx:245: error: aggregate `_stat buffer_in' has incomplete
type
   and cannot be defined
ssgLoadTexture.cxx:245: error: aggregate `_stat buffer_out' has incomplete
type
   and cannot be defined
ssgLoadTexture.cxx:247: error: invalid use of undefined type `struct _stat'
ssgLoadTexture.cxx:245: error: forward declaration of `struct _stat'
ssgLoadTexture.cxx:247: error: invalid use of undefined type `struct _stat'
ssgLoadTexture.cxx:245: error: forward declaration of `struct _stat'
make[2]: *** [ssgLoadTexture.o] Error 1
make[2]: Leaving directory `/home/IBM User/tar/plib-1.8.5/src/ssg'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/IBM User/tar/plib-1.8.5/src'
make: *** [all-recursive] Error 1
"

I'll appreciate your help.

Regards,
DB


------------------------------------------------------------------------  
------
_______________________________________________
plib-devel mailing list
pli...@li...
https://lists.sourceforge.net/lists/listinfo/plib-devel

Re: [Plib-devel] [Plib-users] PLIB-1.8.5 make errors

[John F. F. <joh...@cy...>]

DB,

	Unfortunately I am not an expert on either SSG or cygwin.  I downloaded 
cygwin a couple of months ago and tried building PLIB in it so that I can 
reproduce your error.  After some adventures with "dos2unix" (I'm trying to 
build the SVN version of PLIB, which I downloaded using a Windows 
TortoiseSVN version), I'm running "configure."

	The root of your problem is a "#define" on line 243 of the offending file. 
 Windows, in the infinite wisdom of Microsoft, decided that "stat" needed 
to be "_stat" and the rest of the world is living with the consequences. 
 An immediate fix to your problem is tochange the "#if" on line 242 of 
"ssgLoadTexture.cxx" to the following:

		#if defined(UL_WIN32) && !defined(UL_CYGWIN)

Unfortunately that doesn't actually solve the problem.  Through a chain of 
inclusions, we get back to "util/ul.h" which defines "UL_CYGWIN" if the 
symbol "__CYGWIN__" is defined.  Apparently the latter symbol is not 
defined, and somebody with more knowledge than I needs to put something in 
the Makefile to define it.

		- John


-----Original Message-----
From:	DB
Sent:	Sunday, December 28, 2008 3:01 AM
To:	pli...@li...
Subject:	[Plib-users] PLIB-1.8.5 make errors

Hi,

I've downloaded the current stable plib-1.8.5 and performed the following 
operations in cygwin:
 ./configure --prefix=/fg-1.9.0
and make.

but something went wrong.
I received the following error:
 "
make[2]: Entering directory `/home/IBM User/tar/plib-1.8.5/src/ssg'
if g++ -DPACKAGE_NAME=\"\" -DPACKAGE_TARNAME=\"\" -DPACKAGE_VERSION=\"\" 
-DPACKAGE_STRING=\"\" -DPACKAGE_BUGREPORT=\"\" -DPACKAGE=\"plib\" 
-DVERSION=\
"1.8.5\" -DSTDC_HEADERS=1 -DHAVE_SYS_TYPES_H=1 -DHAVE_SYS_STAT_H=1 
-DHAVE_STDLIB_H=1 -DHAVE_STRING_H=1 -DHAVE_MEMORY_H=1 -DHAVE_STRINGS_H=1 
-DHAVE_INT
TYPES_H=1 -DHAVE_STDINT_H=1 -DHAVE_UNISTD_H=1 -DWIN32=1 -I. -I. 
-I../../src/sg -I../../src/util    -g -O2 -Wall -MT ssgLoadTexture.o -MD 
-MP -MF ".dep
s/ssgLoadTexture.Tpo" -c -o ssgLoadTexture.o ssgLoadTexture.cxx; \
        then mv -f ".deps/ssgLoadTexture.Tpo" ".deps/ssgLoadTexture.Po"; 
else rm -f ".deps/ssgLoadTexture.Tpo"; exit 1; fi
ssgLoadTexture.cxx: In function `int fileMTimeCmp(const char*, const 
char*)':
ssgLoadTexture.cxx:245: error: aggregate `_stat buffer_in' has incomplete 
type
   and cannot be defined
ssgLoadTexture.cxx:245: error: aggregate `_stat buffer_out' has incomplete 
type
   and cannot be defined
ssgLoadTexture.cxx:247: error: invalid use of undefined type `struct _stat'
ssgLoadTexture.cxx:245: error: forward declaration of `struct _stat'
ssgLoadTexture.cxx:247: error: invalid use of undefined type `struct _stat'
ssgLoadTexture.cxx:245: error: forward declaration of `struct _stat'
make[2]: *** [ssgLoadTexture.o] Error 1
make[2]: Leaving directory `/home/IBM User/tar/plib-1.8.5/src/ssg'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/IBM User/tar/plib-1.8.5/src'
make: *** [all-recursive] Error 1
"

I'll appreciate your help.

Regards,
DB
   

Re: [Plib-devel] [PATCH] delete -> delete []

[John F. F. <joh...@cy...>]

Arrgggh ... sorry about that.

I've put it into SVN.

	- John


-----Original Message-----
From:	Melchior FRANZ 
Sent:	Wednesday, December 17, 2008 6:12 PM
To:	pli...@li...
Subject:	[Plib-devel] [PATCH] delete -> delete []

The puInput class wrongly uses "delete" on displayed_text,
which was allocated with "new char[]" (puInput.cxx:33).
It's mostly a cosmetic problem but upsets debuggers like
valgrind.

m.
 << File: pu.diff; charset = iso 8859-15 >>  << File: ATT00045.txt >>  << File: ATT00046.txt >> 

[Plib-devel] [PATCH] delete -> delete []

[Melchior F. <mf...@us...>]

The puInput class wrongly uses "delete" on displayed_text,
which was allocated with "new char[]" (puInput.cxx:33).
It's mostly a cosmetic problem but upsets debuggers like
valgrind.

m.

Re: [Plib-devel] plib-1.8.5: compiling on OSX 10.5.5

[John F. F. <joh...@cy...>]

Andy,

	Thank you for your patch.  I looked at the offending place in the file (it 
has now been moved down to line 145) and find that it has already been 
modified in the latest SVN version:

#if defined(UL_WIN32) || defined(UL_MAC_OSX)

		- John

-----Original Message-----
From:	Andy Christianson
Sent:	Sunday, November 23, 2008 1:46 PM
To:	pli...@li...
Subject:	[Plib-devel] plib-1.8.5: compiling on OSX 10.5.5


I was not able to compile plib-1.8.5 on OSX due to the fact that uint
is not defined by default on this OS. The following patch allows
plib-1.8.5 to compile:

diff -ur ./src/ssg/ssgLoadFLT.cxx 
../plib-1.8.5-apple/src/ssg/ssgLoadFLT.cxx
--- ./src/ssg/ssgLoadFLT.cxx    2008-03-10 22:06:23.000000000 -0400
+++ ../plib-1.8.5-apple/src/ssg/ssgLoadFLT.cxx  2008-11-23
14:32:25.000000000 -0500
@@ -142,7 +142,7 @@

typedef unsigned char ubyte;

-#ifdef UL_WIN32
+#if defined(UL_WIN32) || defined(__APPLE__)
typedef unsigned short ushort;
typedef unsigned int uint;
#endif

-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's 
challenge
Build the coolest Linux based applications with Moblin SDK & win great 
prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
_______________________________________________
plib-devel mailing list
pli...@li...
https://lists.sourceforge.net/lists/listinfo/plib-devel

[Plib-devel] plib-1.8.5: compiling on OSX 10.5.5

[Andy C. <and...@gm...>]

I was not able to compile plib-1.8.5 on OSX due to the fact that uint
is not defined by default on this OS. The following patch allows
plib-1.8.5 to compile:

diff -ur ./src/ssg/ssgLoadFLT.cxx ../plib-1.8.5-apple/src/ssg/ssgLoadFLT.cxx
--- ./src/ssg/ssgLoadFLT.cxx    2008-03-10 22:06:23.000000000 -0400
+++ ../plib-1.8.5-apple/src/ssg/ssgLoadFLT.cxx  2008-11-23
14:32:25.000000000 -0500
@@ -142,7 +142,7 @@

typedef unsigned char ubyte;

-#ifdef UL_WIN32
+#if defined(UL_WIN32) || defined(__APPLE__)
typedef unsigned short ushort;
typedef unsigned int uint;
#endif

Re: [Plib-devel] netSocket patch for Mac OS X

[Tatsuhiro N. <tat...@gm...>]

Thanks!

-----
Tatsuhiro Nishioka

On Nov 11, 2008, at 1:06 AM, "Fay, John F Dr CTR USAF AFMC 46 SK" <joh...@eg... 
 > wrote:

> Tat,
>
>    It is done.  I just uploaded the changes as change set 2135.
>
> John F. Fay
> Technical Fellow
> Jacobs Technology TEAS Group
> 850-883-1294
>
> -----Original Message-----
> From: Tatsuhiro Nishioka [mailto:tat...@gm...]
> Sent: Sunday, November 09, 2008 10:57 AM
> To: PLIB Developers
> Subject: Re: [Plib-devel] netScoket patch for Mac OS X
>
> Hi,
>
> I need to post another patch on this.
> I've recently made a fresh build of PLIB, having a compilation error.
>
> Here is another patch to fix this problem.
> netSocket.h needs to include <netinet/in.h>, not netSocket.cpp.
>
> Best,
>
> Tat
>
>
>
> --- 
> ----------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's  
> challenge
> Build the coolest Linux based applications with Moblin SDK & win  
> great prizes
> Grand prize is a trip for two to an Open Source event anywhere in  
> the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> plib-devel mailing list
> pli...@li...
> https://lists.sourceforge.net/lists/listinfo/plib-devel