<pre>
Session Start: Sat Nov 19 14:56:15 2011
Session Ident: psupreme
�01[14:56] <BriaN_90> hey do u have time?�01[14:57] <BriaN_90> i need someone to debug with me the CButton�01[14:57] <BriaN_90> something doesn't work properly
[15:01] <psupreme> ok
[15:01] <psupreme> did you merge?
�01[15:01] <BriaN_90> no i didn't merge yet�01[15:01] <BriaN_90> i wanna finish the cbutton and merge�01[15:01] <BriaN_90> i think i have a problem with the draw�01[15:01] <BriaN_90> oh but wait�01[15:01] <BriaN_90> i didn't commit�01[15:01] <BriaN_90> the vxproj�01[15:03] <BriaN_90> ok now it's ready
[15:04] <psupreme> okay, so whats wrong
�01[15:05] <BriaN_90> the first problem�01[15:05] <BriaN_90> it doesn't draw properly in the border�01[15:05] <BriaN_90> i followed the instructions on the webesite�01[15:05] <BriaN_90> website�01[15:06] <BriaN_90> and the other problem is the edit�01[15:06] <BriaN_90> it should remove the brackets�01[15:06] <BriaN_90> when i change�01[15:06] <BriaN_90> button but it doesn't
[15:07] <psupreme> visible means you are selected?
[15:07] <psupreme> the button is selected*
�01[15:07] <BriaN_90> u r talking about the CFrame::visible()?
[15:07] <psupreme> ya
�01[15:08] <BriaN_90> visible should return true�01[15:08] <BriaN_90> if the frame has the border�01[15:08] <BriaN_90> false otherwise�01[15:08] <BriaN_90> i think lol
[15:08] <psupreme> ok
[15:08] <psupreme> first bug fixed
[15:08] <psupreme> and you should offset border accordingly
[15:09] <psupreme> ie. put the text in middle NOT @ 0,0
�01[15:10] <BriaN_90> why !Bordered?
[15:10] <psupreme> if its bordered it needs a bigger height
[15:10] <psupreme> so the border displays properly
[15:10] <psupreme> you just have it reversed
�01[15:11] <BriaN_90> oooh right�01[15:12] <BriaN_90> but�01[15:12] <BriaN_90> if u read�01[15:12] <BriaN_90> For width: Set width to the length of Str + 2 (adding 2 for surrounding brackets) or if the Button is bordered set width to the length of Str + 4 (adding 2 for surrounding brackets and 2 for the borders). For height: Set the height to 1 or if the Button is bordered, set the height to 3.�01[15:12] <BriaN_90> that's what i did�01[15:12] <BriaN_90> uhm
[15:13] <psupreme> okay
[15:13] <psupreme> in draw
[15:13] <psupreme> the button is too small
[15:13] <psupreme> not wide enough to store the brackets
�01[15:14] <BriaN_90> uhm on the website it said that it has to be +2 O.o
[15:14] <psupreme> yeah, put +2 back
[15:14] <psupreme> sry
�01[15:15] <BriaN_90> ok
[15:15] <psupreme> but at the top
[15:15] <psupreme> gotta do
[15:15] <psupreme> width(strlen(Str)+4);
[15:15] <psupreme> in the else
�01[15:16] <BriaN_90> why?
[15:16] <psupreme> so the border is wide enough
�01[15:16] <BriaN_90> in this case both are the same, just the height is different�01[15:16] <BriaN_90> so the first part of the if�01[15:16] <BriaN_90> should be +2?
[15:18] <psupreme> whats the flag to check if its selected?
[15:18] <psupreme> because i think edit is broken
�01[15:18] <BriaN_90> there is no flag�01[15:18] <BriaN_90> which edit?
[15:18] <psupreme> CDialog
�01[15:18] <BriaN_90> CButton?�01[15:18] <BriaN_90> why
[15:19] <psupreme> jas told me
�01[15:19] <BriaN_90> uhm
[15:19] <psupreme> unless he fixed it
�01[15:19] <BriaN_90> there was a bug�01[15:19] <BriaN_90> in the CDialog�01[15:19] <BriaN_90> i wrote yestrday�01[15:19] <BriaN_90> here on irc�01[15:20] <BriaN_90> line 49 was�01[15:20] <BriaN_90> draw(0)�01[15:20] <BriaN_90> where fardad told me that it has to be draw(C_FULL_FRAME)�01[15:20] <BriaN_90> and that was the bug�01[15:20] <BriaN_90> that didn't make work my CValEdit
[15:20] <psupreme> rgr
�01[15:21] <BriaN_90> what's rgr lol
[15:21] <psupreme> roger
�01[15:21] <BriaN_90> lol
[15:21] <psupreme> im going to save my changes to a diff file
[15:21] <psupreme> and commit
[15:23] <psupreme> not without
[15:23] <psupreme> yet
�01[15:23] <BriaN_90> ok let me see�01[15:27] <BriaN_90> ok i fixed it�01[15:27] <BriaN_90> now it works�01[15:27] <BriaN_90> even without the borders
[15:27] <psupreme> yeh me too
[15:27] <psupreme> was about to commit
[15:27] <psupreme> ok
�01[15:27] <BriaN_90> look mine�01[15:27] <BriaN_90> see if it's the same
Session Close: Sat Nov 19 15:30:08 2011
Session Start: Sat Nov 19 15:30:08 2011
Session Ident: psupreme
�01[15:33] <BriaN_90> sorry i got disconnected
�01[15:33] <BriaN_90> did u say anything?
�01[15:33] <BriaN_90> <psupreme> yeh me too
�01[15:33] <BriaN_90> <psupreme> was about to commit
�01[15:33] <BriaN_90> <psupreme> ok
�01[15:33] <BriaN_90> <BriaN_90> look mine
�01[15:33] <BriaN_90> <BriaN_90> see if it's the same
�01[15:33] <BriaN_90> that's what i have last
[15:34] <psupreme> (15:28:52) (psupreme) console.display("[",CFrame::absRow(),CFrame::absCol()-1);
[15:34] <psupreme> (15:28:52) (psupreme) console.display("]",CFrame::absRow(),CFrame::absCol()+width()-4);
[15:34] <psupreme> (15:30:58) (psupreme) change your set
[15:34] <psupreme> (15:31:04) (psupreme) to if(_data) delete ...
�01[15:34] <BriaN_90> i asked him about this�01[15:34] <BriaN_90> and it said�01[15:34] <BriaN_90> that if we display the brackets�01[15:34] <BriaN_90> it's way faster�01[15:34] <BriaN_90> that allocate and deallocate�01[15:34] <BriaN_90> than
[15:35] <psupreme> not allocating and deallocating
[15:35] <psupreme> allocate once
[15:35] <psupreme> a string
[15:35] <psupreme> to hold stringlen+null+2 brackets
�01[15:36] <BriaN_90> and display the entire string?
[15:36] <psupreme> im tryign to whip up an example
�01[15:36] <BriaN_90> ok
[15:42] <psupreme> damn your tab spacing
�01[15:42] <BriaN_90> it's 2 characters�01[15:42] <BriaN_90> that's what he said�01[15:42] <BriaN_90> lol
[15:42] <psupreme> so ugly
�01[15:43] <BriaN_90> i know :(
[15:47] <psupreme> git it workin
[15:47] <psupreme> instead of calling display a bunch of times
[15:47] <psupreme> ((char*)_data)[0] = '[';
[15:47] <psupreme> ((char*)_data)[width()-3] = ']';
�01[15:48] <BriaN_90> but then data�01[15:48] <BriaN_90> will be always that
[15:48] <psupreme> nope
[15:48] <psupreme> just overwrite it if not selected
[15:48] <psupreme> right?
�01[15:49] <BriaN_90> yes but after?�01[15:49] <BriaN_90> we need to fix data again
[15:49] <psupreme> for button, data is never used for anything else but the string of the button
�01[15:50] <BriaN_90> uhm
[15:51] <psupreme> and the set function handles everything for us
[15:51] <psupreme> it allocates strlen+3
[15:51] <psupreme> fills it
[15:51] <psupreme> and its good
�01[15:53] <BriaN_90> so u r saying�01[15:53] <BriaN_90> to reallocate the data�01[15:53] <BriaN_90> in the edit?�01[15:53] <BriaN_90> adding the brackets?
[15:56] <psupreme> he doesnt even have a test for button edit
�01[15:57] <BriaN_90> yes he does�01[15:57] <BriaN_90> test 4
[15:57] <psupreme> im running it
[15:57] <psupreme> you cant edit the data of the button
[15:57] <psupreme> like the text
�01[15:58] <BriaN_90> nope u cant�01[15:58] <BriaN_90> the edit is to get the key�01[15:58] <BriaN_90> that is pressed�01[15:58] <BriaN_90> and switch between buttons
[15:58] <psupreme> weird naming
�01[15:58] <BriaN_90> yes lol�01[15:59] <BriaN_90> btw i have to go now�01[15:59] <BriaN_90> i will finish it tonight hopefully�01[15:59] <BriaN_90> let's see if i can make it work :(
[15:59] <psupreme> i can get it finished for you?
�01[15:59] <BriaN_90> if u want�01[15:59] <BriaN_90> we can split the work on the wiki�01[15:59] <BriaN_90> i don't wanna get the credit�01[15:59] <BriaN_90> for something that i did half
[16:00] <psupreme> ??
�01[16:01] <BriaN_90> i don't wanna get the full mark for it�01[16:01] <BriaN_90> if u did half�01[16:01] <BriaN_90> so we will fix the wiki page�01[16:01] <BriaN_90> gotta go�01[16:02] <BriaN_90> let me know if u do something�01[16:02] <BriaN_90> bye
[16:02] <psupreme> k
[16:41] <psupreme> okay
Session Start: Sat Nov 19 21:41:37 2011
Session Ident: psupreme
�01[22:51] <BriaN_90> hey�01[22:51] <BriaN_90> yes i saw it
[22:52] <psupreme> see what i did
[22:52] <psupreme> and why?
�01[22:52] <BriaN_90> there is a bug�01[22:52] <BriaN_90> the cursor should be�01[22:52] <BriaN_90> under the first letter�01[22:53] <BriaN_90> i can fix it�01[22:53] <BriaN_90> ok�01[22:53] <BriaN_90> i fixed it
[22:54] <psupreme> its in my branch
�01[22:54] <BriaN_90> yes i'm using your code
[22:55] <psupreme> oh ok, thought you didnt update
�01[22:55] <BriaN_90> i did�01[22:55] <BriaN_90> last test�01[22:55] <BriaN_90> and then we can merge i think
[22:55] <psupreme> ok
�01[22:56] <BriaN_90> we can merge directly your CButton�01[22:56] <BriaN_90> it's better than mine
[22:57] <psupreme> you know how it works right?
�01[22:57] <BriaN_90> ok now i fixed your CButton�01[22:57] <BriaN_90> i think yes�01[22:57] <BriaN_90> i have this solution in my idea�01[22:57] <BriaN_90> but
[22:57] <psupreme> i explained it in the set(...)
�01[22:57] <BriaN_90> he told me that displaying directly�01[22:58] <BriaN_90> the []�01[22:58] <BriaN_90> was more efficient�01[22:58] <BriaN_90> but who cares lol
[22:58] <psupreme> Id say this way is faster
[22:58] <psupreme> much more efficient than a function call to the display function
�01[22:59] <BriaN_90> the only thing�01[22:59] <BriaN_90> calling a method form the constructor is slow
[22:59] <psupreme> the set method?
�01[22:59] <BriaN_90> yes�01[22:59] <BriaN_90> it would be better
[22:59] <psupreme> nicer than rewriting the code
�01[22:59] <BriaN_90> if the constructor�01[22:59] <BriaN_90> do exactly the same thing of set�01[22:59] <BriaN_90> without calling it�01[22:59] <BriaN_90> yes is nices
[22:59] <psupreme> I bet the compiler just optimizes that in anyway
�01[23:00] <BriaN_90> but slowe in perfomances�01[23:00] <BriaN_90> yes probably�01[23:00] <BriaN_90> right now we don't need that much speed
[23:00] <psupreme> If the function was called thousands of times, i would agree
�01[23:00] <BriaN_90> so we can leave it like this i suppose
[23:00] <psupreme> but its not really neccisary
�01[23:00] <BriaN_90> yes�01[23:01] <BriaN_90> hm my english sux but�01[23:01] <BriaN_90> Clear any junk if he have any�01[23:01] <BriaN_90> is that right?�01[23:01] <BriaN_90> should that be has?
[23:01] <psupreme> oops
[23:01] <psupreme> should be
[23:01] <psupreme> "if we have any"
�01[23:01] <BriaN_90> ok lol�01[23:01] <BriaN_90> i can fix it�01[23:01] <BriaN_90> ok�01[23:01] <BriaN_90> i think we are ready to merge�01[23:02] <BriaN_90> and see what happen
[23:02] <psupreme> yes
[23:02] <psupreme> ill merge my checkmark
�01[23:02] <BriaN_90> and merge CButton too�01[23:02] <BriaN_90> when u r done�01[23:02] <BriaN_90> i will merge my CValEdit
[23:04] <psupreme> I have to manually merge
[23:04] <psupreme> since i edited some files that should NOT be merged
�01[23:04] <BriaN_90> yes i have to do the same thing�01[23:04] <BriaN_90> i messed up some files i think lol�01[23:04] <BriaN_90> i will merge only�01[23:04] <BriaN_90> the two files�01[23:04] <BriaN_90> CValEdit
[23:04] <psupreme> also, make sure files all lowercase
�01[23:05] <BriaN_90> ye i fixed them
[23:05] <psupreme> too keep it consistent
[23:06] <psupreme> okay, added
[23:07] <psupreme> you merge now
�01[23:07] <BriaN_90> ok�01[23:07] <BriaN_90> oh shit�01[23:07] <BriaN_90> i have to comment my cvaledit first�01[23:07] <BriaN_90> lol
[23:07] <psupreme> we'll have to manually change the vcxproj too
�01[23:07] <BriaN_90> i hate commenting�01[23:07] <BriaN_90> yes
[23:07] <psupreme> you dont have to now
�01[23:07] <BriaN_90> ok i will do it after�01[23:09] <BriaN_90> how can u merge only two files?
[23:09] <psupreme> I copy pasted
�01[23:09] <BriaN_90> oh�01[23:10] <BriaN_90> i thought it would ruin the integrity if i do copy paste
[23:10] <psupreme> try to branch only 2 files
[23:10] <psupreme> nvm
�01[23:10] <BriaN_90> yes let me try first�01[23:12] <BriaN_90> it doesn't work�01[23:13] <BriaN_90> i think we have to copy paste lol
[23:13] <psupreme> yes
[23:13] <psupreme> did hex finish his aswell?
�01[23:14] <BriaN_90> i think�01[23:14] <BriaN_90> yes
[23:17] <psupreme> ok
[23:17] <psupreme> update
[23:17] <psupreme> i added all tests aswell
�01[23:17] <BriaN_90> ok�01[23:17] <BriaN_90> we should update�01[23:17] <BriaN_90> the vcxproj�01[23:17] <BriaN_90> i can do it
Session Close: Sat Nov 19 23:17:55 2011
Session Ident: psupreme
�02[23:18] * Not connected to server
�01[23:19] <BriaN_90> sorry my connection�01[23:19] <BriaN_90> i am updating the vcxproj
[23:19] <psupreme> wait
�01[23:19] <BriaN_90> when jas are going to merge?
[23:20] <psupreme> ok
[23:20] <psupreme> update
[23:20] <psupreme> files should be all lowercase
[23:20] <psupreme> hex's files
�01[23:20] <BriaN_90> ye i was going to do it
[23:20] <psupreme> hex's comments so...... useful?
�01[23:21] <BriaN_90> dunno lol
[23:21] <psupreme> jas had a major bug with his code
[23:21] <psupreme> and he has to redo it
�01[23:21] <BriaN_90> ok
[23:21] <psupreme> some heap corruption bug
�01[23:21] <BriaN_90> omg�01[23:21] <BriaN_90> i hope he can do it�01[23:21] <BriaN_90> btw
[23:21] <psupreme> since he was using all old versions of our code
�01[23:21] <BriaN_90> which test should i include in the vcxproj?
[23:22] <psupreme> and all bugs were still present
[23:22] <psupreme> uh
[23:22] <psupreme> none?
�01[23:22] <BriaN_90> yes sounds fine for me�01[23:22] <BriaN_90> and i have to fix
[23:22] <psupreme> hmm
[23:22] <psupreme> maybe one of them tho
�01[23:22] <BriaN_90> the bug in the CDialog::edit line 49�01[23:22] <BriaN_90> i'm fixing it right now�01[23:23] <BriaN_90> and i will commit the trunk�01[23:23] <BriaN_90> without testers for the moment�01[23:23] <BriaN_90> ok i will use all the test�01[23:24] <BriaN_90> for the moment�01[23:24] <BriaN_90> to check if everything works
[23:24] <psupreme> wont compile, each one has a main( )
�01[23:24] <BriaN_90> lol�01[23:24] <BriaN_90> i just add�01[23:24] <BriaN_90> one at time�01[23:24] <BriaN_90> :P
[23:24] <psupreme> if you want..
�01[23:24] <BriaN_90> try to run�01[23:24] <BriaN_90> test 3�01[23:25] <BriaN_90> it doesn't compile�01[23:25] <BriaN_90> there are problem with the cbutton�01[23:25] <BriaN_90> and ccheckmark
[23:25] <psupreme> can you commit vcxproj
[23:25] <psupreme> yes
�01[23:25] <BriaN_90> ok let me commit
[23:26] <psupreme> ok ill fix the bugs for both classes
�01[23:26] <BriaN_90> ok�01[23:26] <BriaN_90> in the meanwhile i will try to run the other test�01[23:26] <BriaN_90> BUT�01[23:26] <BriaN_90> they won't compile of course lol�01[23:26] <BriaN_90> i'll wait for your bug fixes
[23:27] <psupreme> oh right, i think the cc has to be public
�01[23:27] <BriaN_90> we should post this conversation for the other guys
[23:27] <psupreme> yes
[23:27] <psupreme> CLabel copy ctor has to be publuic
[23:27] <psupreme> i believe
�01[23:27] <BriaN_90> uhm
[23:27] <psupreme> and
[23:27] <psupreme> wow
�01[23:27] <BriaN_90> oh wait
[23:27] <psupreme> what happend to CButton
�01[23:27] <BriaN_90> i didn't commit�01[23:28] <BriaN_90> the clabel�01[23:28] <BriaN_90> the new one�01[23:28] <BriaN_90> uhm
[23:28] <psupreme> ctrl+c, v
�01[23:28] <BriaN_90> yes
[23:28] <psupreme> wow
[23:28] <psupreme> how did Cbutton break..
�01[23:29] <BriaN_90> omg
[23:29] <psupreme> yeh
[23:29] <psupreme> i fixed
�01[23:29] <BriaN_90> ok let me update�01[23:29] <BriaN_90> so wait�01[23:30] <BriaN_90> the copy ctor should be public?�01[23:30] <BriaN_90> yes�01[23:30] <BriaN_90> :D
[23:30] <psupreme> yes
[23:31] <psupreme> okay, button fixed
�01[23:31] <BriaN_90> ok let me run everything�01[23:31] <BriaN_90> and commit
[23:32] <psupreme> and, im going to fix all indenting and var names
[23:32] <psupreme> param names should be camelCase
[23:32] <psupreme> not
[23:32] <psupreme> Camelcaseshit
�01[23:33] <BriaN_90> ok now everything works�01[23:33] <BriaN_90> try�01[23:33] <BriaN_90> i committed�01[23:34] <BriaN_90> now i am going to test all testers
[23:34] <psupreme> for label
[23:34] <psupreme> i think you can do
[23:34] <psupreme> L.row()
[23:34] <psupreme> instead of L.CFrame::row()
�01[23:34] <BriaN_90> i like to use the full name�01[23:34] <BriaN_90> because when we are
[23:34] <psupreme> since its not virtual
�01[23:35] <BriaN_90> when we have many classes inherited�01[23:35] <BriaN_90> it's better to see from which class i'm getting it�01[23:35] <BriaN_90> it's my convention lol
[23:35] <psupreme> too much writing for me
�01[23:36] <BriaN_90> ye i know :P�01[23:37] <BriaN_90> there is a bug�01[23:37] <BriaN_90> in the test 5�01[23:37] <BriaN_90> it is trying to include�01[23:37] <BriaN_90> cveditline.h�01[23:37] <BriaN_90> this file doesn't exist
[23:37] <psupreme> lolwtf
�01[23:38] <BriaN_90> maybe it's a typo�01[23:38] <BriaN_90> i'll fix it
[23:38] <psupreme> okay update
[23:38] <psupreme> fixed label formatting
�01[23:38] <BriaN_90> it should be cvaledit
[23:40] <psupreme> good work on valedit
�01[23:40] <BriaN_90> thanks
[23:41] <psupreme> works nice with fixed test
�01[23:41] <BriaN_90> ok i think everything works fine�01[23:41] <BriaN_90> i committed�01[23:42] <BriaN_90> i removed the testers�01[23:42] <BriaN_90> the project has only the classes, no mains
[23:42] <psupreme> yes thats good
�01[23:42] <BriaN_90> when do u think jas will merge?�01[23:42] <BriaN_90> we need to tag before tomorrow at 15.00�01[23:43] <BriaN_90> if he needs help he can ask�01[23:43] <BriaN_90> i see that he doesn't commit a lot�01[23:43] <BriaN_90> lol
[23:43] <psupreme> he commits whole proj
[23:43] <psupreme> instead of incrimently
�01[23:44] <BriaN_90> we need to increase our revision�01[23:44] <BriaN_90> is too low
[23:44] <psupreme> naw
�01[23:44] <BriaN_90> people have up to 300
[23:44] <psupreme> we're just too good and make less mistakes
�01[23:45] <BriaN_90> no we are not�01[23:45] <BriaN_90> :P
[23:45] <psupreme> :(
�01[23:45] <BriaN_90> ok well i think our work is done for tonight�01[23:45] <BriaN_90> do u want me to post the conversation?
[23:45] <psupreme> if you want
[23:45] <BriaN_90> ok
</pre>
