Meeting 01 - Nov 19

From CDOT Wiki
Revision as of 23:56, 19 November 2011 by Lpasquale1 (talk | contribs) (Created page with '<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 CBut…')
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to: navigation, search
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> in ctor
[15:08] <psupreme> if(Bordered
[15:08] <psupreme> should be !Bordered
[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> if(!CFrame::visible())
[15:13] <psupreme> console.display((char *)_data,CFrame::absRow(),CFrame::absCol());
[15:13] <psupreme> else
[15:13] <psupreme> console.display((char *)_data,CFrame::absRow()+1,CFrame::absCol()+1);
[15:13] <psupreme> should be that
[15:13] <psupreme> but....
[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:21] <psupreme> it works, just need to remove [ ] when chaning
[15:21] <psupreme> changin
[15:22] <psupreme> okay
[15:22] <psupreme> update
[15:22] <psupreme> cbutton_new.cpp
[15:23] <psupreme> works with border
[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:28:52) (psupreme) console.setPos(absRow(),absCol());
[15:34] <psupreme> (15:28:53) (psupreme) that
[15:34] <psupreme> (15:29:21) (psupreme) also
[15:34] <psupreme> (15:29:51) (psupreme) instead of "displaying" brackets, why not just allocate a string 2+stringlen, and put the brackets in the string directly
[15:34] <psupreme> (15:30:04) (psupreme) and overwrite if not selected
[15:34] <psupreme> (15:30:08) (psupreme) to ' '
[15:34] <psupreme> (15:30:56) (psupreme) also
[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> just gotta do something like
[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
[16:41] <psupreme> button 100% complete and working
[20:39] <psupreme> hey
[20:39] <psupreme> you see the updated button i did?
Session Close: Sat Nov 19 21:41:37 2011

Session Start: Sat Nov 19 15:30:08 2011
Session Ident: psupreme

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 Start: 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